Include <signal.h> in <nptl-signals.h>

Message ID mvm386vl13e.fsf@hawking.suse.de
State Committed
Headers

Commit Message

Andreas Schwab Jan. 28, 2015, 9:13 a.m. UTC
  Looking at it again I think it is preferable to include the header in
sysdeps/nptl/allocrtsig.c.  Do you agree?

Andreas.
  

Comments

Roland McGrath Jan. 28, 2015, 7:08 p.m. UTC | #1
> Looking at it again I think it is preferable to include the header in
> sysdeps/nptl/allocrtsig.c.  Do you agree?

I was ambivalent about that when reviewing your patch.
Either way is OK by me.
  

Patch

diff --git a/sysdeps/nptl/allocrtsig.c b/sysdeps/nptl/allocrtsig.c
index 6b1c5b2..7e3c621 100644
--- a/sysdeps/nptl/allocrtsig.c
+++ b/sysdeps/nptl/allocrtsig.c
@@ -16,6 +16,7 @@ 
    License along with the GNU C Library; if not, see
    <http://www.gnu.org/licenses/>.  */
 
+#include <signal.h>
 #include <nptl/pthreadP.h>
 
 /* Up to three special signals might be used privately by libpthread.