On Thu, Oct 22, 2015 at 01:19:37PM +0200, Guido Lerch wrote: > > > >>>> Subject: [PATCH] Fixing build warning about freediving > >>>> > >>>> This just fixes an issue with freediving not being with a switch > >>> statement > >>>> that was a little annoying. > >>>> > >>>> location: profice.c > >>>> function: plot_string > > > > Fixing build warning about freediving > > > > This just fixes an annoying warning that freediving wasn't > > covered in the switch statement. > > > > > > I guess it's a matter of style - I like mine better as it is clear and > > concise (I hope) :-) > > Hmm to me reads more or less the same. English is not my native language and > certainly not as good as yours. > Keep on correcting ... :-)
I will. To explain a bit more. You say "fixes an issue" - but the code isn't wrong, there is no "issue". It's just a silly compiler warning. So we are fixing a warning, not an issue. /D _______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
