Commit graph

430712 commits

Author SHA1 Message Date
Miklos Vajna
1ca4092922 sw reqif-xhtml export: write graphic of OLE object at OLE1 level
[MS-OLEDS] 2.2.5 EmbeddedObject actually has a last Presentation field,
and we didn't write it so far.

Luckily all the info can be extracted from the OLE2 container, so we can
roundtrip the OLE1 wrapper around OLE2 nicely.

Change-Id: I8d4922428dd5beeaae8b6cd4da0575db7014e4a2
Reviewed-on: https://gerrit.libreoffice.org/63667
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
2018-11-20 20:30:07 +01:00
Mike Kaganski
18c3d7c4d2 Tidy up the --help text to properly fit into standard 80-ch terminal
Change-Id: I5b7f503e32c10b5a844817c040ab0dde27bf48f7
Reviewed-on: https://gerrit.libreoffice.org/63666
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2018-11-20 19:38:49 +01:00
Khaled Hosny
4765252777 Don’t split HarfBuzz runs for Japanese text
Change-Id: Ib1b632690ae8491732d893788c9b09ced8e49933
Reviewed-on: https://gerrit.libreoffice.org/63668
Tested-by: Jenkins
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
2018-11-20 19:20:50 +01:00
László Németh
542d17d938 tdf#115319 fix Hungarian article before letter i
in words, for example “az Ipsum”, not “a Ipsum”.

This bug was reported by Gellért Gyuris.

Complete commit 1037e3759b
(tdf#115319 references with Hungarian articles)

Change-Id: If930feb11a0308246d2512f0093bcacdc8675d0b
Reviewed-on: https://gerrit.libreoffice.org/63637
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
2018-11-20 18:18:41 +01:00
Roman Kuznetsov
6a65455788 tdf#121490 Hide item Standard (Viewing Mode) toolbar from menu
Change-Id: I9e3a071b9b5792a627a3c5b478ba16f0de386487
Reviewed-on: https://gerrit.libreoffice.org/63630
Tested-by: Jenkins
Reviewed-by: Heiko Tietze <tietze.heiko@gmail.com>
2018-11-20 16:34:40 +01:00
Caolán McNamara
c03a70770e tdf#121538 index 1 never existed
Change-Id: I8493685ceab81a2bd07a3fc75b371a0f679695fb
Reviewed-on: https://gerrit.libreoffice.org/63651
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2018-11-20 16:29:15 +01:00
Jan-Marek Glogowski
f715d6156e Revert "Fix NSS parallel build"
For whatever reason this now fails on Windows for multiple
users (including myself). Doesn't look like an easy fix is
possible, so this is reverted until I find some more time
to look into it (or give up on it).

This reverts commit baf0d6d65d.

Change-Id: I432637131ba8ac8be614496bd6da0e157372ae94
Reviewed-on: https://gerrit.libreoffice.org/63664
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
Tested-by: Jan-Marek Glogowski <glogow@fbihome.de>
2018-11-20 16:17:09 +01:00
Stephan Bergmann
d5ed903618 Set CppunitTest-related env vars only during CppunitTest
...and not for every target's recipe, once solenv/gbuild/CppunitTest.mk is
included

Change-Id: I710160def23fae5f93c5a67ab25e03fdaa008e00
Reviewed-on: https://gerrit.libreoffice.org/63655
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-11-20 15:49:16 +01:00
Andrea Gelmini
330a709899 Removed executable permission on odt file
Change-Id: Icece4e008b653ce334808e3bee968792c4c9e99c
Reviewed-on: https://gerrit.libreoffice.org/63646
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
2018-11-20 15:33:02 +01:00
Noel Grandin
2b29217c58 clean up HelpLocaleString
to make it easier to read.

And fix the last two fall back cases, where it was assigning a different
value to aLocaleStr than it was returning, which means that it would
return a different value on the second invocation.

Change-Id: I6cac1ddd23b0a6a9e4ae8a0b9e7d94c0d877de39
Reviewed-on: https://gerrit.libreoffice.org/63642
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-11-20 15:25:11 +01:00
Samuel Mehrbrodt
9b3387f57d tdf#121537 Add test
Change-Id: Ifdf46dc79f9b3e2b5ab4c9635619a9d7f598affc
Reviewed-on: https://gerrit.libreoffice.org/63654
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2018-11-20 14:56:12 +01:00
Samuel Mehrbrodt
210fbfbbf8 Actually run CheckXTitle tests
@Test annotation was missing

Change-Id: I9e58026e5e131b18e35ab81d7927905316b58f11
Reviewed-on: https://gerrit.libreoffice.org/63653
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2018-11-20 14:53:21 +01:00
Réka Csékei
4610388a6c tdf#120563 Improve the preview of numbering with relative size
Fixes the preview on Customize tab page

Change-Id: If918522bda573c0f312b04592b4b7de957c9416b
Reviewed-on: https://gerrit.libreoffice.org/63649
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2018-11-20 14:48:58 +01:00
Noel Grandin
5e2322ec84 loplugin:unusedenumconstants improvements
add some unit tests, and improve the heuristics

Change-Id: I95aa97a87e178ce8d506bd245710d0ae66ad08a4
Reviewed-on: https://gerrit.libreoffice.org/63647
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-11-20 14:48:27 +01:00
Noel Grandin
9f0a4ae5de clean up s_bTriedCreation static bool
very weird code, no need for hiding this bool inside a method

Change-Id: I75ee67af32ab012fe5ab5ae766abc56212bb08e6
Reviewed-on: https://gerrit.libreoffice.org/63643
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-11-20 14:47:49 +01:00
Sabri Unal
85123ef1ce Related: tdf#121134: Changing http to https where possible
Change-Id: I99f5ce24743a14a78d24da8c64bc6f9adfda6c4c
Reviewed-on: https://gerrit.libreoffice.org/63615
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2018-11-20 14:04:30 +01:00
Noel Grandin
a87ba81099 remove bogus static variable in reportdesign OSectionWindow
which arrived with
   commit bee746960f
   Date:   Tue Feb 1 13:42:47 2011 +0100
    dba34d: limit invalidate calls
and looks extremely dodgy, because it means only the very first report
design window will be correctly configured.

Change-Id: I127c733faaddcacd8507fa74aebe3b6da761327f
Reviewed-on: https://gerrit.libreoffice.org/63641
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2018-11-20 13:10:48 +01:00
Matthias Seidel
dbd5b255f0 Center graphic
(cherry picked from commit 58a04e692ad903a9a16e258afc7f80a37fa188cf)

Change-Id: I3004f9607d9551e9c8bce1bedcfe8426ed392170
Reviewed-on: https://gerrit.libreoffice.org/63609
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2018-11-20 12:54:05 +01:00
Caolán McNamara
0d4d93649d don't need to use a OUStringBuffer for one character
Change-Id: I301a27edf47588d11080b91507b79be177a6f99c
Reviewed-on: https://gerrit.libreoffice.org/63608
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2018-11-20 12:53:49 +01:00
andreas kainz
d2d3dfe1d0 tdf#119714 add Normal View switch in calc pagebreak view
Change-Id: I3b00ee5c851df368a98c6afa0b4c466fc8970fed
Reviewed-on: https://gerrit.libreoffice.org/63566
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-20 12:40:27 +01:00
Samuel Mehrbrodt
4355137901 tdf#121537 Display SuggestedSaveAsName in title if set
Change-Id: I762eb7766a5cbe788c0a360c8a6f37b9b1106412
Reviewed-on: https://gerrit.libreoffice.org/63639
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2018-11-20 12:16:35 +01:00
Serge Krot
f2f2143316 tdf#121456: sw: DOCX: tabs export and TABS_RELATIVE_TO_INDENT prop
In DOCX, w:pos specifies the position of the current custom tab stop
with respect to the current page margins.
But in ODT, zero position could be page margins or paragraph indent
according to used settings DocumentSettingId::TABS_RELATIVE_TO_INDENT

Added new unit test.

Change-Id: Ic56a8527380b6562f4239df1edce0c9b4649af24
Reviewed-on: https://gerrit.libreoffice.org/63460
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2018-11-20 11:55:29 +01:00
Ilhan Yesil
c444eb7934 tdf#121497 "Save As": File Format Type unchanged in Windows
Added checks for filter title, so filter titles are Windows
conform.

Change-Id: I42485c465ea58540a99b8300c5205b902cb86a92
Reviewed-on: https://gerrit.libreoffice.org/63573
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2018-11-20 11:54:25 +01:00
Mike Kaganski
5e8bdd9203 poppler dropped GBool since 0.71
See https://lists.freedesktop.org/archives/libreoffice/2018-November/081410.html

Change-Id: I258e08894486a925bed50a3a4232b6e805af6784
Reviewed-on: https://gerrit.libreoffice.org/63625
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-11-20 11:46:14 +01:00
Stephan Bergmann
b0e819bbc0 Fix loplugin:staticconstfield
...(ExprWithCleanups around the CXXConstructExpr in initializers for members of
O[U]String type, with older Clang, as used in
compilerplugins/clang/test/staticconstfield.cxx), and thus revert
e3e8d52625 "Temporarily disable
compilerplugins/clang/test/staticconstfield" again.

Change-Id: Ic5fcdd1a26e4a6810369e4f9d909200d25feb12e
Reviewed-on: https://gerrit.libreoffice.org/63628
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2018-11-20 11:18:45 +01:00
andreas kainz
7c0c1524f4 NB draw groupedbar compact big update and sync with impress
Change-Id: I69ee7466d6e13d3b0c51c27a8668d567c0427a35
Reviewed-on: https://gerrit.libreoffice.org/63562
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-20 11:08:57 +01:00
Gabor Kelemen
a87a062fd4 tdf#42949 Fix IWYU warnings in sc/qa/{perf,unit}/*
Found with bin/find-unneeded-includes
Only removal proposals are dealt with here.

Change-Id: I031faa67192f21d20938c8cc8056db138326a1ca
Reviewed-on: https://gerrit.libreoffice.org/63143
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2018-11-20 10:42:40 +01:00
andreas kainz
228fed36ab tdf#120918 change AutoFormat Style in Colibre
Change-Id: I23956b671a24cc8ac85cec4ea6220d1107f99b1e
Reviewed-on: https://gerrit.libreoffice.org/63624
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-20 10:26:08 +01:00
Jim Raykowski
697c308431 tdf#121401 Keyboard navigation for Writer NB contextual single
Change-Id: Ibcb56c6c0f1456c472061039ecc0835928846724
Reviewed-on: https://gerrit.libreoffice.org/63458
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-20 10:24:58 +01:00
Jens Carl
dc0616cda9 tdf#45904 Move XEnumeration Java tests to C++
Move XEnumeration Java test to C++
for ScIndexEnumeration_SubTotalFieldsEnumeration.

Change-Id: Ibe2b283e99230c5e517eb80858ff31459f2c9844
Reviewed-on: https://gerrit.libreoffice.org/63627
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>
2018-11-20 10:18:46 +01:00
andreas kainz
511c2d6b28 Sifr icon theme rework svg files
additional changes will come

Change-Id: Ia7f8c7d7b3c1471a4c851d1e362f1b67ef8e413c
Reviewed-on: https://gerrit.libreoffice.org/63055
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-20 10:08:19 +01:00
andreas kainz
976b7930a1 ZoomMode string description is now PopupLabel Label is now shorter
Change-Id: I442c4bf9acbd85f340371014c20be3292a016beb
Reviewed-on: https://gerrit.libreoffice.org/63613
Tested-by: Jenkins
Reviewed-by: Heiko Tietze <tietze.heiko@gmail.com>
2018-11-20 09:36:11 +01:00
Jan-Marek Glogowski
6cdfe5ebb4 Drop some headless mode variants
This fixes OSX "make debugrun" by dropping VCL_HIDE_WINDOWS
handling and removing the internal GetPseudoHeadless() API.

While at it moves the DialogCancelMode enum out of Application.

Change-Id: I4876e752ddbfc39dd44faa673fb0e97810089a75
Reviewed-on: https://gerrit.libreoffice.org/61598
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2018-11-20 09:19:09 +01:00
Miklos Vajna
d220e476df tdf#120551 DOCX import: fix position of group shape with absolute position
Regression from commit af313fc149
(tdf#105143 DOCX import: enable DoNotCaptureDrawObjsOnPage layout compat
option, 2017-01-06), the group shape which covers most of the single
page in the document had a negative left position, while it should have
a small positive one (checking the drawingML markup).

This was a pre-existing problem, but now it's visible since we no longer
force objects to be on the page in Word compat mode.

Seeing the ODT import never positions group shapes (it's just a
container for child shapes), probably the DOCX import shouldn't do that
either. Start moving into this direction, first only for
absolute-positioned toplevel group shapes only, which already fixes the
bug.

Change-Id: I152ba06a81a2bd09195a4c724da4b8878b0457bb
Reviewed-on: https://gerrit.libreoffice.org/63606
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2018-11-20 09:07:44 +01:00
Jens Carl
0c627e4940 tdf#45904 Move XEnumeration Java tests to C++
Move XEnumeration Java tests to C++
for ScIndexEnumeration_SpreadsheetViewPanesEnumeration.

Change-Id: I03c05cd894bf84816618ac28deb01ecb2fd8779e
Reviewed-on: https://gerrit.libreoffice.org/63622
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>
2018-11-20 08:59:37 +01:00
Jan-Marek Glogowski
baf0d6d65d Fix NSS parallel build
This is just a minimal solution and obviously not fully parallel.
We still have to walk the tree twice:
1. copy all needed headers to dist (export export_private)
2. the parallel build (libs)

The fixes are:
* Add a few rm and @$(MAKE_OBJDIR) calls to various targets
* Serialize minimal steps in nss_build_all
  => Build libs of nss/lib and nss/cmd/lib before all other
  => drop the minimal all target
* Make $(SHARED_LIBRARY) depend on $(LIBRARY), otherwise dll lib
  creation on Windows builds a static lib, instead of the dll one
* Drop all "export:: private_export" rules. We run this before
  the build and duplicate targets break the header install

Change-Id: I1c35985c90d2d63990e04a0d2bb9d8ae921a2182
Reviewed-on: https://gerrit.libreoffice.org/62011
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2018-11-20 08:57:20 +01:00
Jens Carl
20a7d7af61 tdf#45904 Move XEnumeration Java tests to C++
Move XEnumeration Java tests to C++
for ScIndexEnumeration_SpreadsheetsEnumeration.

Change-Id: Id6fbc4dab54e2b50be42df7ade912a5aaf1cb97c
Reviewed-on: https://gerrit.libreoffice.org/63621
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>
2018-11-20 08:45:17 +01:00
andreas kainz
ebe76aa262 tdf#95073 new document Breeze icon without gray background
Change-Id: I5d0d44be038f61616fe71cbfecf492b5abffbc8a
Reviewed-on: https://gerrit.libreoffice.org/63612
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-20 07:59:34 +01:00
andreas kainz
562f550ef1 tdf#120300 add setminima icons to colibre and breeze
Change-Id: Ie454dab3f23bee4bf0321fe6f5a657a57cc5ef5e
Reviewed-on: https://gerrit.libreoffice.org/63611
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-20 07:59:23 +01:00
Jens Carl
49907ec311 tdf#45904 Move XEnumeration Java tests to C++
Move XEnumeration Java tests to C++
for ScIndexEnumeration_TableAutoFormatEnumeration.

Change-Id: Ibe9f4cffaecc4f3fada04f82425031e6773b32d7
Reviewed-on: https://gerrit.libreoffice.org/63619
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>
2018-11-20 07:00:02 +01:00
Jens Carl
f0ddfe64f3 tdf#45904 Move XEnumeration Java tests to C++
Move XEnumeration Java tests to C++
for ScIndexEnumeration_SheetCellRangesEnumeration.

Change-Id: Ic3bc6f38244c4f476d29826accc397b4b6902f31
Reviewed-on: https://gerrit.libreoffice.org/63605
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>
2018-11-20 06:59:51 +01:00
Jens Carl
242ea8eea8 tdf#45904 Move XEnumeration Java tests to C++
Move XEnumeration Java tests to C++
for ScIndexEnumeration_FunctionDescriptionEnumeration.

Change-Id: I6c15abff293996a6bac8e4a6c0004853fae13990
Reviewed-on: https://gerrit.libreoffice.org/63604
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>
2018-11-20 06:53:15 +01:00
Adolfo Jayme Barrientos
ef2dfe4d54 HIG fixes in asciifilterdialog.ui
Change-Id: Ie54292794aa4ecfce3fed97102aef75d0fec2d33
Reviewed-on: https://gerrit.libreoffice.org/63614
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2018-11-20 03:37:49 +01:00
Adolfo Jayme Barrientos
8b011be70b Make Persona selection dialog look less busy
• No need for a horizontal line to be randomly there.
• Get rid of GtkFrames that enclose only one element.
  Seriously, they are cancer. Use them properly!
• … and while I’m here, use a stock label for the Search button.
  I noticed the associated help page not using the “Go” name.

Change-Id: Id196d8d970a219285c5cc606ab9b4327cdce2e91
Reviewed-on: https://gerrit.libreoffice.org/62238
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2018-11-20 01:36:28 +01:00
Bjoern Michaelsen
b14d30f8bf no more SwClient/SwModify for SwTextTableRows
Change-Id: I267904c751c9b32b9b64b0cf1b7f9d922bc7e1f7
Reviewed-on: https://gerrit.libreoffice.org/63529
Tested-by: Jenkins
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@libreoffice.org>
2018-11-19 22:45:08 +01:00
andreas kainz
295059082e Colibre Breeze icons: add masterpage and pagemode icons
for draw from impress

Change-Id: I60bb28ddf47ca4aa3ef16c9fd61e26ac80d4e721
Reviewed-on: https://gerrit.libreoffice.org/63563
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-19 22:16:38 +01:00
andreas kainz
68814c91eb tdf#61580 add insert comment to writer and impress
Change-Id: I00fa63f9cb7c31f54732446aaf62b4a9b9fce522
Reviewed-on: https://gerrit.libreoffice.org/63565
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2018-11-19 22:16:27 +01:00
Caolán McNamara
2cb9c55e4d Resolves: tdf#112696 survive missing SwDrawContact
its removed when the object was removed from the page to be reinserted
as as a group member

Change-Id: I8edeb89ae07b73572b65413817212a79ef8957b3
Reviewed-on: https://gerrit.libreoffice.org/63578
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2018-11-19 22:05:33 +01:00
Jens Carl
4e467724fc tdf#45904 Move XEnumeration Java tests to C++
Move XEnumeration Java tests to C++
for ScIndexEnumeration_DatabaseRangesEnumeration.

Change-Id: I33fac410b4b05f88dad45f859ae466fabb8303fc
Reviewed-on: https://gerrit.libreoffice.org/63592
Tested-by: Jenkins
Reviewed-by: Jens Carl <j.carl43@gmx.de>
2018-11-19 21:54:41 +01:00
Stephan Bergmann
e3e8d52625 Temporarily disable compilerplugins/clang/test/staticconstfield
Of the "Gerrit Linux clang/dbgutil" build nodes, one (tb79-pollux) has done the
lode update to use Clang 3.9.1, while the other two nodes (tb75-lilith and
tb76-maggie) are still at the old Clang 3.8.0.  When building Gerrit changes for
master, tb79-pollux started to fail in
compilerplugins/clang/test/staticconstfield now
(<https://ci.libreoffice.org/job/gerrit_linux_clang_dbgutil/20247/> and
<https://ci.libreoffice.org/job/gerrit_linux_clang_dbgutil/20252/>) while such
builds on the other two nodes still succeed.  So it looks like Clang 3.9.1 has
some issue with that loplugin, which will need further inspection.  For now, to
not break Gerrit/Jenkins builds, disable that test.

Change-Id: I1bd380fb98d3b9a411dd04578a5620cdb27cd35b
2018-11-19 21:06:55 +01:00