Messages in this thread | | | Date | Mon, 2 Nov 2015 08:20:26 +0100 (CET) | From | Julia Lawall <> | Subject | Re: [PATCH] ALSA: firewire-lib: fix compare_const_fl.cocci warnings |
| |
On Mon, 2 Nov 2015, Takashi Sakamoto wrote:
> Hi, > > On Nov 02 2015 15:55, Julia Lawall wrote: > > Move constant to the right of binary operators. > > > > Generated by: scripts/coccinelle/misc/compare_const_fl.cocci > > > > CC: Takashi Sakamoto <o-takashi@sakamocchi.jp> > > Signed-off-by: Fengguang Wu <fengguang.wu@intel.com> > > Signed-off-by: Julia Lawall <julia.lawall@lip6.fr> > > --- > > > > Depends on personal taste, but the modified version does look a little > > nicer to me, since one can better see the relationship between the fdf > > and sfc fields. > > > > amdtp-am824.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > --- a/sound/firewire/amdtp-am824.c > > +++ b/sound/firewire/amdtp-am824.c > > @@ -36,7 +36,7 @@ int amdtp_am824_set_parameters(struct am > > if (err < 0) > > return err; > > > > - s->fdf = AMDTP_FDF_AM824 | s->sfc; > > + s->fdf = s->sfc | AMDTP_FDF_AM824; > > > > /* > > * In IEC 61883-6, one data block represents one event. In ALSA, one > > Could I request explainations about the advantage of this modification? I > cannot imagine good reasons for this subtle changes...
I put the reason under the --- (better see the relationship between fdf and sfc). But if you think the code makes more sense as it is, just ignore the patch.
julia
| |