[devel] gcc4&multisync0.90

[devel] gcc4&multisync0.90

[Alexey Shabalin]

День добрый
помогите собрать multisync0.90
gcc3.4 собирает, а gcc4 ругается на следующий участок.

===============================
static void msync_fs_make_config(fs_options *options, char **data, int *size)
{
        xmlDocPtr doc;

        doc = xmlNewDoc((xmlChar*)"1.0");
        doc->children = xmlNewDocNode(doc, NULL, (xmlChar*)"config", NULL);

        xmlNewChild(doc->children, NULL, (xmlChar*)"path",
(xmlChar*)options->path);
        xmlNewChild(doc->children, NULL, (xmlChar*)"recursive",
(xmlChar*)(options->recursive ? "TRUE" : "FALS

        xmlDocDumpMemory(doc, (xmlChar **)data, size);
        *size++;
}
===================================
skip
===================================
msync_fs_make_config(options, &config, &size);


file_sync.c: In function 'msync_fs_make_config':
file_sync.c:129: warning: value computed is not used

file_sync.c:129 - это   *size++;

Если убрать * - ругань исчезает. коректно ли это?



-- 
Alexey Shabalin

Re: [devel] gcc4&multisync0.90

[Sergey N. Yatskevich]

В Пнд, 29/05/2006 в 16:26 +0400, Alexey Shabalin пишет:
> День добрый
> помогите собрать multisync0.90
> gcc3.4 собирает, а gcc4 ругается на следующий участок.
> 
> ===============================
> static void msync_fs_make_config(fs_options *options, char **data, int *size)
> {
>         xmlDocPtr doc;
> 
>         doc = xmlNewDoc((xmlChar*)"1.0");
>         doc->children = xmlNewDocNode(doc, NULL, (xmlChar*)"config", NULL);
> 
>         xmlNewChild(doc->children, NULL, (xmlChar*)"path",
> (xmlChar*)options->path);
>         xmlNewChild(doc->children, NULL, (xmlChar*)"recursive",
> (xmlChar*)(options->recursive ? "TRUE" : "FALS
> 
>         xmlDocDumpMemory(doc, (xmlChar **)data, size);
>         *size++;
> }
> ===================================
> skip
> ===================================
> msync_fs_make_config(options, &config, &size);
> 
> 
> file_sync.c: In function 'msync_fs_make_config':
> file_sync.c:129: warning: value computed is not used
> 
> file_sync.c:129 - это   *size++;
> 
> Если убрать * - ругань исчезает. коректно ли это?
Нет, не корректно. Всё дело в приоритете операций разыменования
указателя (*) и увеличения на единицу (++).
Если развернуть данную конструкцию (*size++), то получится
следующее:

int* tmp = size;
size = size + 1;
*tmp; // вот здесь и ругается

Судя по коду здесь просто к вычисленному xmlDocDumpMemory размеру
добавляется единица (наверное для того, чтобы учесть завершающий
'\0'), так что правильно должно быть так: (*size)++

-- 
Sergey N. Yatskevich <syatskevich@altlinux.ru>

Re: [devel] gcc4&multisync0.90

[Alexey Shabalin]

Спасибо.
я уже опробовал
*size = *size +1; - работает


29.05.06, Sergey N. Yatskevich<syatskevich@altlinux.ru> написал(а):
> В Пнд, 29/05/2006 в 16:26 +0400, Alexey Shabalin пишет:
> > День добрый
> > помогите собрать multisync0.90
> > gcc3.4 собирает, а gcc4 ругается на следующий участок.
> >
> > ===============================
> > static void msync_fs_make_config(fs_options *options, char **data, int *size)
> > {
> >         xmlDocPtr doc;
> >
> >         doc = xmlNewDoc((xmlChar*)"1.0");
> >         doc->children = xmlNewDocNode(doc, NULL, (xmlChar*)"config", NULL);
> >
> >         xmlNewChild(doc->children, NULL, (xmlChar*)"path",
> > (xmlChar*)options->path);
> >         xmlNewChild(doc->children, NULL, (xmlChar*)"recursive",
> > (xmlChar*)(options->recursive ? "TRUE" : "FALS
> >
> >         xmlDocDumpMemory(doc, (xmlChar **)data, size);
> >         *size++;
> > }
> > ===================================
> > skip
> > ===================================
> > msync_fs_make_config(options, &config, &size);
> >
> >
> > file_sync.c: In function 'msync_fs_make_config':
> > file_sync.c:129: warning: value computed is not used
> >
> > file_sync.c:129 - это   *size++;
> >
> > Если убрать * - ругань исчезает. коректно ли это?
> Нет, не корректно. Всё дело в приоритете операций разыменования
> указателя (*) и увеличения на единицу (++).
> Если развернуть данную конструкцию (*size++), то получится
> следующее:
>
> int* tmp = size;
> size = size + 1;
> *tmp; // вот здесь и ругается
>
> Судя по коду здесь просто к вычисленному xmlDocDumpMemory размеру
> добавляется единица (наверное для того, чтобы учесть завершающий
> '\0'), так что правильно должно быть так: (*size)++
>
> --
> Sergey N. Yatskevich <syatskevich@altlinux.ru>
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel


-- 
Alexey Shabalin