Skip to content

C
core
Dal (branch)/etiket değiştir
  1. 30 Nis, 2019 2 kayıt (commit)
    • Noel Grandin's avatar
      revert part of "improve loplugin:stringconstant" · e02b9ccf
      Noel Grandin yazdı 
      Revert part of
    commit dd8d5e57
    Date:   Mon Apr 29 11:18:21 2019 +0200
    improve loplugin:stringconstant

sberg's original gerrit comment:
but there can also be other problematic overloads for parameters like
`void const *` or `std::string_view`.  I'm not sure this change is worth
the potential false positives.

and continuing IRC discussion:
<noelgrandin> I'll revert the compilerplugins/ part
<sberg> noelgrandin, my main concern is that /if/ somebody eventually
runs into such an overload situation, it's really hard to get the
warnings/errors fixed for those people, short of going into the plugin
itself

Change-Id: I4916ce8943c4319d7ef9084e22d6a0eeb430b15c
      e02b9ccf
    • Noel Grandin's avatar
      improve loplugin:stringconstant · dd8d5e57
      Noel Grandin yazdı 
      to find more places we can elide the OUString() constructor at call
sites

Change-Id: Ie09f3c61f2c4b4959c97dc98ebcbaf7c51d5d713
Reviewed-on: https://gerrit.libreoffice.org/71514
Tested-by: Jenkins
Reviewed-by: 's avatarNoel Grandin <noel.grandin@collabora.co.uk>
      dd8d5e57
  3. 29 Nis, 2019 1 kayıt (commit)
  5. 29 Kas, 2018 1 kayıt (commit)
  7. 27 Kas, 2018 1 kayıt (commit)
  9. 14 Agu, 2018 1 kayıt (commit)
  11. 10 Agu, 2018 1 kayıt (commit)
  13. 06 Tem, 2018 1 kayıt (commit)
    • Michael Stahl's avatar
      compilerplugins: try to make these work with icecream · ff002524
      Michael Stahl yazdı 
      There are some problems here, this should fix one of them: the
getFilename function returns "<stdin>" for spelling locations, because
the input to clang is sort of preprocessed via -frewrite-includes if
icecream is used and the file is built on a remote host (whereas it's
apparently not preprocessed if the file is compiled locally by icecream).

Using getPresumedLoc() uses the #line directives in the preprocessed
input, which avoids the problem but is more expensive, so try to use it
only when necessary.

The getFileEntry(getMainFileID())->getName() pattern will also result
in "<stdin>", but fortunately icecream passes -main-file-name,
which oddly enough isn't used by the SourceManager's spelling locations,
but is available separately via CodeGenOptions.

This builds everything successfully with clang version 6.0.0:
ICECC_PREFERRED_HOST=myremote make check gb_SUPPRESS_TESTS=t

Change-Id: Ic121511683e5302d7b9d85186c8b9c4a5443fa1b
Reviewed-on: https://gerrit.libreoffice.org/54993
Tested-by: Jenkins
Reviewed-by: 's avatarThorsten Behrens <Thorsten.Behrens@CIB.de>
      ff002524
  15. 29 Ock, 2018 1 kayıt (commit)
  17. 07 Kas, 2017 1 kayıt (commit)
  19. 28 Eyl, 2017 1 kayıt (commit)
  21. 27 Eyl, 2017 1 kayıt (commit)
  23. 22 Eyl, 2017 2 kayıt (commit)
  25. 18 May, 2017 1 kayıt (commit)
  27. 11 Şub, 2017 1 kayıt (commit)
  29. 10 Şub, 2017 1 kayıt (commit)
  31. 02 Şub, 2017 2 kayıt (commit)
  33. 01 Şub, 2017 3 kayıt (commit)
  35. 31 Ock, 2017 1 kayıt (commit)
  37. 28 Ock, 2017 1 kayıt (commit)
  39. 27 Ock, 2017 1 kayıt (commit)
  41. 25 Ock, 2017 1 kayıt (commit)
  43. 11 Ock, 2017 3 kayıt (commit)
  45. 10 Ock, 2017 1 kayıt (commit)
  47. 18 Ara, 2016 2 kayıt (commit)
  49. 14 Ara, 2016 1 kayıt (commit)
  51. 25 Kas, 2016 1 kayıt (commit)
  53. 30 Agu, 2016 1 kayıt (commit)
  55. 29 Agu, 2016 1 kayıt (commit)
  57. 29 Haz, 2016 3 kayıt (commit)
  59. 28 Haz, 2016 1 kayıt (commit)