Re: Add FALLTHRU to gimple-ssa-sprintf.c

2016-09-21 Thread Martin Sebor
On 09/21/2016 05:07 AM, Marek Polacek wrote: On Wed, Sep 21, 2016 at 12:57:23PM +0200, Jakub Jelinek wrote: On Wed, Sep 21, 2016 at 12:52:33PM +0200, Marek Polacek wrote: Pointed out by Tobias. This looks like a missing fallthru marker. Ok? 2016-09-21 Marek Polacek * gimple-ssa-s

Re: Add FALLTHRU to gimple-ssa-sprintf.c

2016-09-21 Thread Jakub Jelinek
On Wed, Sep 21, 2016 at 01:07:20PM +0200, Marek Polacek wrote: > On Wed, Sep 21, 2016 at 12:57:23PM +0200, Jakub Jelinek wrote: > > On Wed, Sep 21, 2016 at 12:52:33PM +0200, Marek Polacek wrote: > > > Pointed out by Tobias. This looks like a missing fallthru marker. > > > > > > Ok? > > > > > > 2

Re: Add FALLTHRU to gimple-ssa-sprintf.c

2016-09-21 Thread Marek Polacek
On Wed, Sep 21, 2016 at 12:57:23PM +0200, Jakub Jelinek wrote: > On Wed, Sep 21, 2016 at 12:52:33PM +0200, Marek Polacek wrote: > > Pointed out by Tobias. This looks like a missing fallthru marker. > > > > Ok? > > > > 2016-09-21 Marek Polacek > > > > * gimple-ssa-sprintf.c (pass_sprintf_

Re: Add FALLTHRU to gimple-ssa-sprintf.c

2016-09-21 Thread Jakub Jelinek
On Wed, Sep 21, 2016 at 12:52:33PM +0200, Marek Polacek wrote: > Pointed out by Tobias. This looks like a missing fallthru marker. > > Ok? > > 2016-09-21 Marek Polacek > > * gimple-ssa-sprintf.c (pass_sprintf_length::compute_format_length): > Add falls through comment. This is o