diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-05-14 17:14:18 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-06-11 11:38:15 +0200 |
| commit | 1f08bff31238d5818c54a0b86570689644dff087 (patch) | |
| tree | d4d6f4b62a3c48ddeb85ba89818247c17f2578c8 /include/editeng/outliner.hxx | |
| parent | ff130af9661a57d290dbf89b54a4c0ce8d0f71ea (diff) | |
new loplugin:shouldreturnbool
look for methods returning only 1 and/or 0, which (most of the time)
should be returning bool.
Off by default, because some of this is a matter of taste
Change-Id: Ib17782e629888255196e89d4a178618a9612a0de
Reviewed-on: https://gerrit.libreoffice.org/54379
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'include/editeng/outliner.hxx')
| -rw-r--r-- | include/editeng/outliner.hxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/include/editeng/outliner.hxx b/include/editeng/outliner.hxx index 6ad87b3385e5..3c0126f3b31b 100644 --- a/include/editeng/outliner.hxx +++ b/include/editeng/outliner.hxx @@ -248,8 +248,7 @@ public: void CreateSelectionList (std::vector<Paragraph*> &aSelList) ; - // Returns the number of selected paragraphs - sal_Int32 Select( Paragraph const * pParagraph, bool bSelect = true); + void Select( Paragraph const * pParagraph, bool bSelect = true); OUString GetSelected() const; void SelectRange( sal_Int32 nFirst, sal_Int32 nCount ); |