Commit Graph

7 Commits

Author SHA1 Message Date
Stephan Bergmann
0626e66d76 Avoid loplugin:indentation after preproc conditional inclusion lines
...to not trigger (for --disable-dbgutil) at

>                         rContentAtPos.eContentAtPos = IsAttrAtPos::TableBoxFml;
> #ifdef DBG_UTIL
>                         if( RES_BOXATR_VALUE == pItem->Which() )
>                             rContentAtPos.eContentAtPos = IsAttrAtPos::TableBoxValue;
>                         else
> #endif
>                             const_cast<SwTableBoxFormula*>(static_cast<const SwTableBoxFormula*>(pItem))->PtrToBoxNm( &pTableNd->GetTable() );

(sw/source/core/crsr/crstrvl.cxx:1710)

Change-Id: I8fd5c8a1cd40450bdd8cc97057e5233a0134f044
Reviewed-on: https://gerrit.libreoffice.org/68030
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-02-20 11:14:55 +01:00
Stephan Bergmann
177747c445 SourceManager::getExpansionRange already returns CharSourceRange since Clang 7
Change-Id: Ic7c6c648c71203116ca074bd7392a48ff850cd51
Reviewed-on: https://gerrit.libreoffice.org/67864
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>
2019-02-15 12:19:34 +01:00
Andrea Gelmini
6d52cd1c0b Fix typo
Change-Id: I036a41bd4d46973a8b6fb51a90c6506cf3f9b8f0
Reviewed-on: https://gerrit.libreoffice.org/67748
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2019-02-13 05:07:43 +01:00
Stephan Bergmann
a072b15e96 More loplugin:indentation Qt "emit" hackery
Change-Id: I6f71546a5f6ab07bb4df6fc89213962e74984c84
2019-02-12 08:54:19 +01:00
Noel Grandin
c04eb27aff remove debugging code
Change-Id: Ibb6f3a1c711959cb4f32541dba96f005e51c9040
2019-02-11 11:37:07 +02:00
Noel Grandin
8c0bb7692a loplugin:indentation workaround qt macros
Change-Id: I17bb0b462bb4ce7e8128c9ad33491aa8b691e566
2019-02-11 10:59:35 +02:00
Noel Grandin
f76bf262a6 new loplugin indentation
look for mixed indentation in compound statements, which makes them hard
to read, and sometimes makes it look like a statement is associated with
a nearby if/for

Change-Id: Ic8429cee1f9a86d938097a4a8769a2bce97b3361
Reviewed-on: https://gerrit.libreoffice.org/63283
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-02-11 08:24:19 +01:00