diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-08-14 08:28:13 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-08-14 09:14:39 +0200 |
| commit | 0c3bd1d482ece1f2217cc87069179161f6a5bb56 (patch) | |
| tree | 86bab8ab624fd60076838f068f010372321a204b /compilerplugins/clang/test/droplong.cxx | |
| parent | 11535357df4e4ec828435229ff784b9868ee583a (diff) | |
new loplugin:droplong
The types 'long' and 'unsigned long' are different sizes on different
platforms, making them wholy unsuitable for portable code.
And when I mean different sizes, I mean 64bit Linux and 64bit Windows
have different bit sizes.
Change-Id: Id4d8cd709bf71a3f85ed6515fae2b0a21ae8c25d
Reviewed-on: https://gerrit.libreoffice.org/41130
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'compilerplugins/clang/test/droplong.cxx')
| -rw-r--r-- | compilerplugins/clang/test/droplong.cxx | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/compilerplugins/clang/test/droplong.cxx b/compilerplugins/clang/test/droplong.cxx new file mode 100644 index 000000000000..c38984342c24 --- /dev/null +++ b/compilerplugins/clang/test/droplong.cxx @@ -0,0 +1,20 @@ +/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4; fill-column: 100 -*- */ +/* + * This file is part of the LibreOffice project. + * + * This Source Code Form is subject to the terms of the Mozilla Public + * License, v. 2.0. If a copy of the MPL was not distributed with this + * file, You can obtain one at http://mozilla.org/MPL/2.0/. + */ + + +int main() +{ + int x = 1; + int y = 1; + long tmp = x + y; // expected-error {{rather replace long with 'int' [loplugin:droplong]}} + (void)tmp; + tmp = x + y; // expected-error {{rather replace long with 'int' [loplugin:droplong]}} +} + +/* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */ |