[Bug libc/4000] New: <argp.h> has a buggy __NTH macro definition

[Bug libc/4000] New: <argp.h> has a buggy __NTH macro definition

[madcoder at debian dot org]

well not strictly "buggy" but a differnt one from <sys/cdefs.h> which is 
rather surprising.

Here is a patche to have the same one as in <sys/cdefs.h>.
Note that maybe #including <sys/cdefs.h> is better, I'm not very used to such 
things yet.

-- 
           Summary: <argp.h> has a buggy __NTH macro definition
           Product: glibc
           Version: unspecified
            Status: NEW
          Severity: minor
          Priority: P2
         Component: libc
        AssignedTo: drepper at redhat dot com
        ReportedBy: madcoder at debian dot org
                CC: glibc-bugs at sources dot redhat dot com


http://sourceware.org/bugzilla/show_bug.cgi?id=4000

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/4000] <argp.h> has a buggy __NTH macro definition

[madcoder at debian dot org]

------- Additional Comments From madcoder at debian dot org  2007-02-07 23:59 -------
Created an attachment (id=1545)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=1545&action=view)
copy __NTH (and __THROW) definition from <sys/cdefs.h>


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4000

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/4000] <argp.h> has a buggy __NTH macro definition

[jakub at redhat dot com]

------- Additional Comments From jakub at redhat dot com  2007-02-12 12:37 -------
sys/cdefs.h is already included when you include argp.h (stdio.h includes
features.h which includes sys/cdefs.h), the define is there
just for the case where argp.h is used standalone as part of gnulib on non-glibc
system.
Your patch only makes things worse, no header should ever undef these macros.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4000

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/4000] <argp.h> has a buggy __NTH macro definition

[madcoder at debian dot org]

------- Additional Comments From madcoder at debian dot org  2007-02-12 14:42 -------
(In reply to comment #2)
> sys/cdefs.h is already included when you include argp.h (stdio.h includes
> features.h which includes sys/cdefs.h), the define is there
> just for the case where argp.h is used standalone as part of gnulib on
> non-glibc system.
> Your patch only makes things worse, no header should ever undef these
> macros.

okay that makes sense. thanks for the explanation, I suppose you can close the 
bug then (i'll even do it).

Cheers,

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |INVALID


http://sourceware.org/bugzilla/show_bug.cgi?id=4000

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.