Re: [devel] [git update] packages/rpmdb-utils: heads/master

Re: [devel] [git update] packages/rpmdb-utils: heads/master

[Slava Semushin]

2007/7/3, Alexey Gladkov <legion / altlinux.org>:
[...]
> diff --git a/rpmdb-utils/rpmdb2binlist.c b/rpmdb-utils/rpmdb2binlist.c
> index 3a0b861..54726d2 100644
> --- a/rpmdb-utils/rpmdb2binlist.c
> +++ b/rpmdb-utils/rpmdb2binlist.c
> @@ -18,12 +18,15 @@
>   ***** END LICENSE BLOCK ******/
>
>  #include <stdio.h>
> +#include <string.h>
>  #include <sys/types.h>
>  #include <sys/stat.h>
>  #include <fcntl.h>
> +#include <assert.h>
[...]

А зачем был добавлен хедер string.h? В добавленных в этом коммите
строках ни одна ф-ция из string.h не используется. Я посмотрел весь
файл и не нашел использования ф-ций из этого хедера.

На 99% уверен, что этот хедер здесь не нужен. А вот stdlib.h напротив
отсутствует, но нужен для ф-ции exit() и констант
EXIT_{SUCCESS,FAILURE}

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [git update] packages/rpmdb-utils: heads/master

[Alexey Gladkov]

Slava Semushin пишет:
> А зачем был добавлен хедер string.h? В добавленных в этом коммите
> строках ни одна ф-ция из string.h не используется. Я посмотрел весь
> файл и не нашел использования ф-ций из этого хедера.

Да. string.h тут лишний.

> На 99% уверен, что этот хедер здесь не нужен. А вот stdlib.h напротив
> отсутствует, но нужен для ф-ции exit() и констант
> EXIT_{SUCCESS,FAILURE}

А вот тут вы не правы. Этот хэдер добавляется внутри. Иначе я узнал бы
об этом при сборке.

-- 
Rgrds, legion

Re: [devel] [git update] packages/rpmdb-utils: heads/master

[Slava Semushin]

05.07.07, Alexey Gladkov<legion / altlinux.ru> написал(а):
[...]
> > На 99% уверен, что этот хедер здесь не нужен. А вот stdlib.h напротив
> > отсутствует, но нужен для ф-ции exit() и констант
> > EXIT_{SUCCESS,FAILURE}
>
> А вот тут вы не правы. Этот хэдер добавляется внутри. Иначе я узнал бы
> об этом при сборке.

Внутри это где? Он скорее всего неявно подключается в stdio.h, к примеру.


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [git update] packages/rpmdb-utils: heads/master

[Stanislav Ievlev]

On Fri, Jul 06, 2007 at 11:50:50AM +0700, Slava Semushin wrote:
> 05.07.07, Alexey Gladkov<legion / altlinux.ru> написал(а):
> [...]
> > > На 99% уверен, что этот хедер здесь не нужен. А вот stdlib.h напротив
> > > отсутствует, но нужен для ф-ции exit() и констант
> > > EXIT_{SUCCESS,FAILURE}
> >
> > А вот тут вы не правы. Этот хэдер добавляется внутри. Иначе я узнал бы
> > об этом при сборке.
> 
> Внутри это где? Он скорее всего неявно подключается в stdio.h, к примеру.
По любому от явного повторного включения хуже никому не будет.

Re: [devel] [git update] packages/rpmdb-utils: heads/master

[Igor Zubkov]

В сообщении от Friday 06 July 2007 09:49:27 Stanislav Ievlev написал(а):
> On Fri, Jul 06, 2007 at 11:50:50AM +0700, Slava Semushin wrote:
> > 05.07.07, Alexey Gladkov<legion / altlinux.ru> написал(а):
> > [...]
> >
> > > > На 99% уверен, что этот хедер здесь не нужен. А вот stdlib.h напротив
> > > > отсутствует, но нужен для ф-ции exit() и констант
> > > > EXIT_{SUCCESS,FAILURE}
> > >
> > > А вот тут вы не правы. Этот хэдер добавляется внутри. Иначе я узнал бы
> > > об этом при сборке.
> >
> > Внутри это где? Он скорее всего неявно подключается в stdio.h, к примеру.
>
> По любому от явного повторного включения хуже никому не будет.

Кстати, как я понимаю в свежий gcc 4.2 не соберёт такой код. Я просто не явное 
включение инклюдов. Так что, в любом случае лучше добавить хедер. Меньше 
проблем будет в будущем.

-- 
icesik