Displaying 4 results from an estimated 4 matches for "fix_whatev".
Did you mean:
fix_whatever
2020 Oct 19
5
[RFC] treewide: cleanup unreachable breaks
...way.
>
> Sending us a patch, without even a diffstat to review, isn't going to
> get you very far...
Tom,
If you're able to automate this cleanup, I suggest checking in a
script that can be run on a directory. Then for each subsystem you
can say in your commit "I ran scripts/fix_whatever.py on this subdir."
Then others can help you drive the tree wide cleanup. Then we can
enable -Wunreachable-code-break either by default, or W=2 right now
might be a good idea.
Ah, George (gbiv@, cc'ed), did an analysis recently of
`-Wunreachable-code-loop-increment`, `-Wunreachable-co...
2020 Oct 19
0
[RFC] treewide: cleanup unreachable breaks
...tch, without even a diffstat to review, isn't going to
> > get you very far...
>
> Tom,
> If you're able to automate this cleanup, I suggest checking in a
> script that can be run on a directory. Then for each subsystem you
> can say in your commit "I ran scripts/fix_whatever.py on this subdir."
> Then others can help you drive the tree wide cleanup. Then we can
> enable -Wunreachable-code-break either by default, or W=2 right now
> might be a good idea.
I remember using clang-modernize in the past to fix issues very
similar to this, if clang machiner...
2020 Oct 17
10
[RFC] treewide: cleanup unreachable breaks
From: Tom Rix <trix at redhat.com>
This is a upcoming change to clean up a new warning treewide.
I am wondering if the change could be one mega patch (see below) or
normal patch per file about 100 patches or somewhere half way by collecting
early acks.
clang has a number of useful, new warnings see
2020 Oct 17
10
[RFC] treewide: cleanup unreachable breaks
From: Tom Rix <trix at redhat.com>
This is a upcoming change to clean up a new warning treewide.
I am wondering if the change could be one mega patch (see below) or
normal patch per file about 100 patches or somewhere half way by collecting
early acks.
clang has a number of useful, new warnings see