Change in core[master]: sal_Bool => bool if internal use only (in sw/source/core/crs...

Gerrit gerrit at gerrit.libreoffice.org
Mon Jul 9 03:10:19 PDT 2012


>From Björn Michaelsen <bjoern.michaelsen at canonical.com>:

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 <ruderphilipp at gmail.com>
Gerrit-Reviewer: Björn Michaelsen <bjoern.michaelsen at canonical.com>



More information about the LibreOffice mailing list