[devel] Q: snprintf

[devel] Q: snprintf

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 552 bytes --]

Здравствуйте.

Нижеследующий кусок (одного патча) меня озадачил.  Существует ли
надежный способ определить: была ли переменная объевлена как имя
массива, или как указатель?

--- 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


[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: snprintf

[Alexander Bokovoy]

On Sun, Mar 21, 2004 at 10:39:23PM +0300, Alexey Tourbin wrote:
> Здравствуйте.
> 
> Нижеследующий кусок (одного патча) меня озадачил.  Существует ли
> надежный способ определить: была ли переменная объевлена как имя
> массива, или как указатель?
В каком языке? В C?
-- 
/ Alexander Bokovoy
Samba Team                      http://www.samba.org/
ALT Linux Team                  http://www.altlinux.org/
Midgard Project Ry              http://www.midgard-project.org/

[devel] Re: Q: snprintf

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1217 bytes --]

On Mon, Mar 22, 2004 at 05:53:12AM +0200, Alexander Bokovoy wrote:
> > Нижеследующий кусок (одного патча) меня озадачил.  Существует ли
> > надежный способ определить: была ли переменная объявлена как имя
> > массива, или как указатель?
> В каком языке? В C?

Да.  Вот смотри:

On Sun, Mar 21, 2004 at 10:39:23PM +0300, Alexey Tourbin 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))

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Re: Q: snprintf

[Alexander Bokovoy]

On Mon, Mar 22, 2004 at 07:21:19PM +0300, Alexey Tourbin wrote:
> On Mon, Mar 22, 2004 at 05:53:12AM +0200, Alexander Bokovoy wrote:
> > > Нижеследующий кусок (одного патча) меня озадачил.  Существует ли
> > > надежный способ определить: была ли переменная объявлена как имя
> > > массива, или как указатель?
> > В каком языке? В C?
> 
> Да.  Вот смотри:
> 
> On Sun, Mar 21, 2004 at 10:39:23PM +0300, Alexey Tourbin 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))
Криво, честно говоря. Но в целом, наверное, сойдет.

-- 
/ Alexander Bokovoy
Samba Team                      http://www.samba.org/
ALT Linux Team                  http://www.altlinux.org/
Midgard Project Ry              http://www.midgard-project.org/

[devel] Re: Q: snprintf

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1291 bytes --]

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().  Мечтать не вредно...

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]