strl* (was: Re: Advent4 patch, second try)

Nanomad condellog at gmail.com
Fri Sep 28 11:43:25 CEST 2007


On 9/27/07, Luis Javier Merino <ljmerino at gmail.com> wrote:
> #define GROW_SIZE       1000
[CUT]
>                 *nsize = ((slen + GROW_SIZE) / GROW_SIZE) * GROW_SIZE;
[CUT]
>                         *nsize = ((*size + slen + GROW_SIZE - 1) /
> GROW_SIZE) * GROW_SIZE;
[CUT]

Just a quick comment: dividing and then multipling for the same value
just does nothing (or, better, can lead to problems since you are
working with int) (or am I missing something?)
-- 
Official Ubuntu GNU/Linux Translator
Currently Running:
Ubuntu "Gutsty Gibbon" on Desktop PC (P4 1.4Ghz)
Ubuntu "Feisty Fawn" on Laptop (Centrino 2Ghz)



More information about the dslinux-devel mailing list