[Gluster-devel] useless if-before-free and no-semantic-change patches

Vijay Bellur vbellur at redhat.com
Thu Jul 12 10:17:50 UTC 2012


On 07/12/2012 12:52 PM, Jim Meyering wrote:
> Hi,
>

>
> In fact, while it's handy for review/rebase/regen to keep them in
> separate c-sets, I'd be happy to merge the 6 automatically-induced
> c-sets into one, if desired.  Let me know.


I think it would be handy to have a single changeset as they are related 
in a broader sense.

>
> Also, for a change like this that is mostly stylistic,
> (sort of like dead-code elimination), is it appropriate
> to use the static-analysis BUG: number, or is there another one?
>

It might be a good idea to have a new bug for committing this patch set.

Thanks,
Vijay






More information about the Gluster-devel mailing list