r290773 - Wdocumentation fix
Simon Pilgrim via cfe-commits
cfe-commits at lists.llvm.org
Sat Dec 31 02:24:15 PST 2016
On 30/12/2016 23:12, Richard Smith wrote:
> On 30 Dec 2016 3:06 pm, "Simon Pilgrim via cfe-commits"
> <cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>> wrote:
> Author: rksimon
> Date: Fri Dec 30 16:55:33 2016
> New Revision: 290773
> URL: http://llvm.org/viewvc/llvm-project?rev=290773&view=rev
> Wdocumentation fix
> Thanks, but please be careful you don't introduce trailing whitespace
> changes to unrelated code in the future.
Sorry about that - I'm working on my alt laptop for the holidays and
didn't have the same whitespace diff settings for tortoisesvn -
shouldn't happpen again.
> Also perhaps we should turn this warning on by default for regular
> selfhosted builds so we don't need to wait for a buildbot to find
> issues like this?
The buildbots are there for us to unearth these kind of things,
otherwise (in extremis) you'd be asking everyone to test on a much
broader range of compile tools than they perhaps have access to.
IMO the problem is that these mundane warnings tend to stay unnoticed in
build logs, where people don't go unless the build goes red - I would be
interested in hearing people's thoughts on either enabling Werror by
default in CMake gen and/or enabling Werror on a wider range of
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits