Re: [cfe-users] [run-clang-tidy] new replacement overlaps with an existing replacement

2018-12-10 Thread Jonas Toth via cfe-users
Hi Mateusz, comments inline. Am 10.12.18 um 22:14 schrieb Mateusz Loskot via cfe-users: > Hi, > > I'm running clang-tidy 7.0 (also tried 5.0) to modernise some aspects > of Boost.GIL (https://github.com/boostorg/gil) source code. > > I've noticed, clang-tidy 7.0 (also 5.0) does not apply fixes fo

[cfe-users] [run-clang-tidy] new replacement overlaps with an existing replacement

2018-12-10 Thread Mateusz Loskot via cfe-users
Hi, I'm running clang-tidy 7.0 (also tried 5.0) to modernise some aspects of Boost.GIL (https://github.com/boostorg/gil) source code. I've noticed, clang-tidy 7.0 (also 5.0) does not apply fixes for some of modernize-use-* checks, especially modernize-use-using. I run it this way: ``` cd ${BOOS

Re: [cfe-users] My compiler crashes .... but the online godbolt one doesn't ?!

2018-12-10 Thread Jonas Toth via cfe-users
The snippet you show is already quite minimal, so its ok to file another bug specifically for that one. Am 10.12.18 um 19:17 schrieb David Blaikie via cfe-users: > Possible that the online one isn't built with assertions enabled (you > could test this with other known crashers to see if they repro

Re: [cfe-users] My compiler crashes .... but the online godbolt one doesn't ?!

2018-12-10 Thread David Blaikie via cfe-users
Possible that the online one isn't built with assertions enabled (you could test this with other known crashers to see if they reproduce on godbolt with assertion crash dumps or only with raw segfaults)? If that's the case,t hen it's possible that the codepath that should assert continues on and pe