Sorry!

It's in pan-calendar.c that function.

I should have checked gcc flags before asking!

Sorry again and thanks for the answer!

2009/6/17 Laurent Monin <z...@norz.org>

> Oscar Miras a écrit :
> > Hi all,
> >
> Hi Oscar,
> > Sometimes in geeqie code I find things like this:
> > In pan-util.c ; pan_calendar_computer function
> >
> No such function in pan-util.c.
> Though there's a pan_calendar_compute(),  in pan-calendar.c at line 189
> (current trunk).
> > dt -= 60 * 60 * 24;
> >
> > That could be substituted by a constant ( 86400).
> > I suppose that's left that way to help programmers to understand the
> code.
> True.
> > But it implies CPU time to do the calculation.
> False.
>
> > Maybe compiler when reaches that line, automatically do the
> > calculating, so in execution time that instruction is always processed
> > as dt -= 86400. I don't know.
> >
> True. You can verify by yourself examing generated assembler code (ie.
> using gcc -S).
> > Should I leave that way, or now that I'm 'doxygenating' some code,
> > could I substitute it for dt -= 86400 /** 60 * 60 * 24 */  ?
> >
> >
> Useless in this case.
> Keep it as is, it is readable and doesn't need further optimization.
>
>
> Please do not attempt to over-optimize, or if needed, please use tools
> to measure execution speed changes.
>
>
> Regards,
>
> --
> Zas
>
>
> ------------------------------------------------------------------------------
> Crystal Reports - New Free Runtime and 30 Day Trial
> Check out the new simplified licensing option that enables unlimited
> royalty-free distribution of the report engine for externally facing
> server and web deployment.
> http://p.sf.net/sfu/businessobjects
> _______________________________________________
> Geeqie-devel mailing list
> Geeqie-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geeqie-devel
>
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Reply via email to