[PATCH 5/5] Limit the scope of gasidx in add_plan_to_notes

Anton Lundin glance at acc.umu.se
Thu Jul 10 13:34:24 PDT 2014


On 10 July, 2014 - Dirk Hohndel wrote:

> On Thu, Jul 10, 2014 at 10:11:46PM +0200, Anton Lundin wrote:
> > Signed-off-by: Anton Lundin <glance at acc.umu.se>
> > ---
> >  planner.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/planner.c b/planner.c
> > index 785db4d..5cd9ba9 100644
> > --- a/planner.c
> > +++ b/planner.c
> > @@ -518,7 +518,7 @@ static unsigned int *sort_stops(int *dstops, int dnr, struct gaschanges *gstops,
> >  static void add_plan_to_notes(struct diveplan *diveplan, struct dive *dive, bool show_disclaimer)
> >  {
> >  	char buffer[20000], temp[1000];
> > -	int len, gasidx, lastdepth = 0, lasttime = 0;
> > +	int len, lastdepth = 0, lasttime = 0;
> >  	struct divedatapoint *dp = diveplan->dp;
> >  	bool gaschange = !plan_verbatim;
> >  	struct divedatapoint *nextdp = NULL;
> > @@ -637,7 +637,7 @@ static void add_plan_to_notes(struct diveplan *diveplan, struct dive *dive, bool
> >  
> >  	snprintf(temp, sizeof(temp), "%s", translate("gettextFromC", "Gas consumption:"));
> >  	len += snprintf(buffer + len, sizeof(buffer) - len, "</tbody></table></div><div><br>%s<br>", temp);
> > -	for (gasidx = 0; gasidx < MAX_CYLINDERS; gasidx++) {
> > +	for (int gasidx = 0; gasidx < MAX_CYLINDERS; gasidx++) {
> 
> IIRC we don't want to use this in C files because some older C compilers
> throw up with it...
> 

I checked, and we already do, for windows atleast =)

windows.c:		for (int i = 0; i < len; i++)


//Anton


-- 
Anton Lundin	+46702-161604


More information about the subsurface mailing list