[devel] union in C99

[devel] union in C99

[Led]

потребовалось сделать что-то типа такого:
-    int         i_bframe_adaptive;
+    union {
+        int         i_bframe_adaptive;
+        int         b_bframe_adaptive __attribute__((__deprecated__));
+    };

это поля structx 264_param_t . Но при -std=c99 получаем:
libx264.c:165: error: 'x264_param_t' has no member named 'i_bframe_adaptive'

Как это сейчас правильно реализуется?

-- 
Led

Re: [devel] union in C99

[Damir Shayhutdinov]

> потребовалось сделать что-то типа такого:
> -    int         i_bframe_adaptive;
> +    union {
> +        int         i_bframe_adaptive;
> +        int         b_bframe_adaptive __attribute__((__deprecated__));
> +    };
>
> это поля structx 264_param_t . Но при -std=c99 получаем:
> libx264.c:165: error: 'x264_param_t' has no member named 'i_bframe_adaptive'
>
> Как это сейчас правильно реализуется?

То, что ты хочешь сделать - называется anonymous union

Re: [devel] union in C99

[Led]

On Tuesday, 23 September 2008 19:26:24 Damir Shayhutdinov wrote:
> > потребовалось сделать что-то типа такого:
> > -    int         i_bframe_adaptive;
> > +    union {
> > +        int         i_bframe_adaptive;
> > +        int         b_bframe_adaptive __attribute__((__deprecated__));
> > +    };
> >
> > это поля structx 264_param_t . Но при -std=c99 получаем:
> > libx264.c:165: error: 'x264_param_t' has no member named
> > 'i_bframe_adaptive'
> >
> > Как это сейчас правильно реализуется?
>
> То, что ты хочешь сделать - называется anonymous union

Да, это я знаю. Но, похоже, в C99 это не допускается. Нашёл, что для gcc это 
обходится путём добавления __extension__ перед union. Может есть более 
корректный и переносимый способ реализации подобного?

-- 
Led

Re: [devel] union in C99

[Led]

On Tuesday, 23 September 2008 20:13:37 Konstantin Pavlov wrote:
> 2008/9/23 Led <ledest@gmail.com>
>
> > потребовалось сделать что-то типа такого:
> > -    int         i_bframe_adaptive;
> > +    union {
> > +        int         i_bframe_adaptive;
> > +        int         b_bframe_adaptive __attribute__((__deprecated__));
> > +    };
> >
> > это поля structx 264_param_t . Но при -std=c99 получаем:
> > libx264.c:165: error: 'x264_param_t' has no member named
> > 'i_bframe_adaptive'
>
> Мне кажется, легче запатчить клиентов libx264, чем менять ее API и дальше.

Запатчить надо, конечно. Но компат-хук я пока сделаю - при сборке warning про 
deprecated будет (если мейнтейнеры, конечно, обращают внимание на warning'и).

-- 
Led

Re: [devel] union in C99

[Kirill A. Shutemov]

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

On Tue, Sep 23, 2008 at 07:22:41PM +0300, Led wrote:
> потребовалось сделать что-то типа такого:
> -    int         i_bframe_adaptive;
> +    union {
> +        int         i_bframe_adaptive;
> +        int         b_bframe_adaptive __attribute__((__deprecated__));
> +    };
> 
> это поля structx 264_param_t . Но при -std=c99 получаем:
> libx264.c:165: error: 'x264_param_t' has no member named 'i_bframe_adaptive'
> 
> Как это сейчас правильно реализуется?

Как вариант, можно попробовать что-то типа такого:

-    int         i_bframe_adaptive;
+#define i_bframe_adaptive bframe_adaptive.i
+#define i_bframe_adaptive bframe_adaptive.b
+    union {
+        int         i;
+        int         b __attribute__((__deprecated__));
+    } bframe_adaptive;

-- 
Regards,  Kirill A. Shutemov
 + Belarus, Minsk
 + ALT Linux Team, http://www.altlinux.com/

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

Re: [devel] union in C99

[Led]

On Tuesday, 23 September 2008 21:22:47 Kirill A. Shutemov wrote:
> On Tue, Sep 23, 2008 at 07:22:41PM +0300, Led wrote:
> > потребовалось сделать что-то типа такого:
> > -    int         i_bframe_adaptive;
> > +    union {
> > +        int         i_bframe_adaptive;
> > +        int         b_bframe_adaptive __attribute__((__deprecated__));
> > +    };
> >
> > это поля structx 264_param_t . Но при -std=c99 получаем:
> > libx264.c:165: error: 'x264_param_t' has no member named
> > 'i_bframe_adaptive'
> >
> > Как это сейчас правильно реализуется?
>
> Как вариант, можно попробовать что-то типа такого:
>
> -    int         i_bframe_adaptive;
> +#define i_bframe_adaptive bframe_adaptive.i
> +#define i_bframe_adaptive bframe_adaptive.b
> +    union {
> +        int         i;
> +        int         b __attribute__((__deprecated__));
> +    } bframe_adaptive;

Спасибо. Да, такой вариант я тоже рассматривал, но с define'ами не очень 
красиво получается - в этом случае будет тупо подменяться все 
i_bframe_adaptive/b_bframe_adaptive, даже если это не поля структуры 
x264_param_t. Пока остановился на варианте:
+#ifdef __GNUC__
+    __extension__ union {
+        int         i_bframe_adaptive;
+        int         b_bframe_adaptive __attribute__((__deprecated__));
+    };
+#else
     int         i_bframe_adaptive;
+#endif

-- 
Led