[devel] Re: Q: snprintf

Alexey Tourbin at на altlinux.ru
Вт Мар 23 03:01:21 MSK 2004


On Mon, Mar 22, 2004 at 06:24:50PM +0200, Alexander Bokovoy wrote:
> > > --- src/lvm.c.orig     2004-01-13 11:37:53.000000000 -0200
> > > +++ src/lvm.c  2004-01-13 11:33:34.000000000 -0200
> > > @@ -32,7 +32,7 @@
> > > 
> > >  /* function to convert a lua_Number to a string */
> > >  #ifndef lua_number2str
> > > -#define lua_number2str(s,n)     sprintf((s), LUA_NUMBER_FMT, (n))
> > > +#define lua_number2str(s,n)     snprintf((s), sizeof((s)), LUA_NUMBER_FMT, (n))
> > >  #endif
> > > 
> > 
> > Т.е. есть кусок некоторого API, которое broken by design, но его
> > кому-то хочется зафиксить.  Но в данном случае использование sizeof
> > в макросе некорректно, т.к. переменная s может быть указателем, а не
> > именем массива, а размер любого указателя всегда равен sizeof(long).
> > 
> > Пока приходит в голову только следующее:
> > 
> > #define lua_number2str(s,n)	(sizeof(s) > sizeof(char*)) ? \
> > 	snprintf((s), sizeof((s)), LUA_NUMBER_FMT, (n)) : \
> > 	sprintf((s), LUA_NUMBER_FMT, (n))
> Криво, честно говоря. Но в целом, наверное, сойдет.

На самом деле это не часть API. :)
Так что достаточно проверить src/lvm.c.

В целом же, обобщенный оператор sizeof() должен быть RTTI функцией,
которая умеет залезать в структуры malloc().  Мечтать не вредно...
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : отсутствует
Тип     : application/pgp-signature
Размер  : 189 байтов
Описание: отсутствует
Url     : http://lists.altlinux.org/pipermail/devel/attachments/20040323/f39dd2d7/attachment.bin 


Подробная информация о списке рассылки Devel