* Re: [devel] [git update] packages/rpmdb-utils: heads/master @ 2007-07-05 10:04 ` Slava Semushin 2007-07-05 13:47 ` Alexey Gladkov 0 siblings, 1 reply; 5+ messages in thread From: Slava Semushin @ 2007-07-05 10:04 UTC (permalink / raw) To: ALT Devel discussion list 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [git update] packages/rpmdb-utils: heads/master 2007-07-05 10:04 ` [devel] [git update] packages/rpmdb-utils: heads/master Slava Semushin @ 2007-07-05 13:47 ` Alexey Gladkov 2007-07-06 4:50 ` Slava Semushin 0 siblings, 1 reply; 5+ messages in thread From: Alexey Gladkov @ 2007-07-05 13:47 UTC (permalink / raw) To: ALT Devel discussion list Slava Semushin пишет: > А зачем был добавлен хедер string.h? В добавленных в этом коммите > строках ни одна ф-ция из string.h не используется. Я посмотрел весь > файл и не нашел использования ф-ций из этого хедера. Да. string.h тут лишний. > На 99% уверен, что этот хедер здесь не нужен. А вот stdlib.h напротив > отсутствует, но нужен для ф-ции exit() и констант > EXIT_{SUCCESS,FAILURE} А вот тут вы не правы. Этот хэдер добавляется внутри. Иначе я узнал бы об этом при сборке. -- Rgrds, legion ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [git update] packages/rpmdb-utils: heads/master 2007-07-05 13:47 ` Alexey Gladkov @ 2007-07-06 4:50 ` Slava Semushin 2007-07-06 6:49 ` Stanislav Ievlev 0 siblings, 1 reply; 5+ messages in thread From: Slava Semushin @ 2007-07-06 4:50 UTC (permalink / raw) To: ALT Devel discussion list 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [git update] packages/rpmdb-utils: heads/master 2007-07-06 4:50 ` Slava Semushin @ 2007-07-06 6:49 ` Stanislav Ievlev 2007-07-06 12:51 ` Igor Zubkov 0 siblings, 1 reply; 5+ messages in thread From: Stanislav Ievlev @ 2007-07-06 6:49 UTC (permalink / raw) To: ALT Devel discussion list 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, к примеру. По любому от явного повторного включения хуже никому не будет. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [git update] packages/rpmdb-utils: heads/master 2007-07-06 6:49 ` Stanislav Ievlev @ 2007-07-06 12:51 ` Igor Zubkov 0 siblings, 0 replies; 5+ messages in thread From: Igor Zubkov @ 2007-07-06 12:51 UTC (permalink / raw) To: ALT Devel discussion list В сообщении от 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2007-07-06 12:51 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2007-07-05 10:04 ` [devel] [git update] packages/rpmdb-utils: heads/master Slava Semushin 2007-07-05 13:47 ` Alexey Gladkov 2007-07-06 4:50 ` Slava Semushin 2007-07-06 6:49 ` Stanislav Ievlev 2007-07-06 12:51 ` Igor Zubkov
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git