>From Björn Michaelsen <[email protected]>: Björn Michaelsen has posted comments on this change.
Change subject: sal_Bool => bool if internal use only (in sw/source/core/crsr/) ...................................................................... Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) looking good in general .................................................... File sw/source/core/crsr/findfmt.cxx Line 36: bool bSrchForward = (fnMove == fnMoveForward); While at it this bool can also be const ... -- To view, visit https://gerrit.libreoffice.org/276 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iedf20880edaa454d55c45c70ed0e6dce3fa90623 Gerrit-PatchSet: 1 Gerrit-Project: core Gerrit-Branch: master Gerrit-Owner: Philipp Riemer <[email protected]> Gerrit-Reviewer: Björn Michaelsen <[email protected]> _______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
