Review Request 113430: More warnings fixes

Alex Merry kde at randomguy3.me.uk
Fri Oct 25 14:08:54 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113430/
-----------------------------------------------------------

Review request for KDE Frameworks.


Repository: kdelibs


Description
-------

This is a series of commits:


warnings--: parentheses around '&&' within '||'


Fix a couple of warnings inadventently caused by 139a09da


Use a better hack for dealing with gperf's unhelpful inlining

gperf (in ANSI-C mode) attempts to inline functions in gnu90 mode, which
*only* generates inline versions.  This causes linker errors if the
compiler decides not to use inlining.

Previously, there was a hack (claiming to prevent warnings) that
disabled __inline keyword, which fixed the linking issue but caused
another warning (about ignored attributes) to be emitted because the
__gnu_inline__ attribute was still there, and that doesn't work without
one of the normal inline keywords.

This version leaves __inline alone, but prevents the use of
__gnu_inline__.


Use KHTML_NO_EXPORT on the decl (not def) of tagPriorityArray

Visibility attributes only work on declarations, not on definitions.


Remove virtual modifier from signal

A virtual signal makes no sense.


Diffs
-----

  khtml/src/html/dtd.cpp 9cfa345af618861a9b8ddf9a93dd180c9a15c6d7 
  khtml/src/html/dtd.h 1e04e58b356c5d0ece23f8fc78cffdc6cf64817c 
  khtml/src/css/parser.y e24d5f46e27c686fa855feee27dbe2dbf39108cb 
  khtml/src/html/htmlparser.cpp 6c23dd5b5a2a1e567adc8cfa0233e10838cf3451 
  khtml/src/rendering/render_canvasimage.cpp 346a65491e459c3d8f1c731340ff5a6ae33b35f1 
  khtml/src/rendering/render_inline.cpp 84e07eab5604e4882fd816826b6e59c317c53805 
  tier1/itemmodels/src/kselectionproxymodel.cpp c7ded3f8c54eba0afa1974ddb67972eddf7d7f1a 
  tier1/solid/src/solid/backends/udev/udevbutton.h 39eb6683fc611dfff5b6fef69e66ab709238e889 

Diff: http://git.reviewboard.kde.org/r/113430/diff/


Testing
-------

Builds, tests pass.


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131025/21dd30d3/attachment.html>


More information about the Kde-frameworks-devel mailing list