Review Request: break_lines.h:137: warning: suggest parentheses around & & within ||

Guy Maurel guy-kde at maurel.de
Wed Oct 21 18:36:01 BST 2009


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

Review request for kdelibs.


Summary
-------

The dashboard
  http://dashboard.akonadi-project.org/CDash/index.php?project=kdelibs
reports:
/.../kdelibs/khtml/rendering/break_lines.h:137: warning: suggest parentheses around && within ||

at
inline bool isBreakable( const QChar *str, const int pos, int len )

my proposal: complete with some more parentheses
see: break_lines-gm-24.diff


Diffs
-----

  /trunk/KDE/kdelibs/khtml/rendering/break_lines.h 1038700 

Diff: http://reviewboard.kde.org/r/1922/diff


Testing
-------


Thanks,

Guy





More information about the kde-core-devel mailing list