From d78c6fe3d3b6097d098c0ea26a6bd722f2b70b42 Mon Sep 17 00:00:00 2001 From: Jennifer Averett Date: Thu, 13 Jan 2000 18:22:32 +0000 Subject: + Comment cleanup --- c/src/exec/posix/src/mqueuecreatesupp.c | 5 +++-- c/src/exec/posix/src/mqueuenametoid.c | 5 +++-- c/src/exec/posix/src/mqueueopen.c | 8 +++++--- c/src/exec/posix/src/semaphorecreatesupp.c | 9 +-------- 4 files changed, 12 insertions(+), 15 deletions(-) (limited to 'c/src/exec/posix/src') diff --git a/c/src/exec/posix/src/mqueuecreatesupp.c b/c/src/exec/posix/src/mqueuecreatesupp.c index ceebb2adc2..148f5b54cb 100644 --- a/c/src/exec/posix/src/mqueuecreatesupp.c +++ b/c/src/exec/posix/src/mqueuecreatesupp.c @@ -31,6 +31,9 @@ /*PAGE * * _POSIX_Message_queue_Create_support + * + * This routine does the actual creation and initialization of + * a poxix message queue. */ int _POSIX_Message_queue_Create_support( @@ -126,8 +129,6 @@ int _POSIX_Message_queue_Create_support( set_errno_and_return_minus_one( ENOSPC ); } - - /* XXX - need Names to be a string!!! */ _Objects_Open( &_POSIX_Message_queue_Information, &the_mq->Object, diff --git a/c/src/exec/posix/src/mqueuenametoid.c b/c/src/exec/posix/src/mqueuenametoid.c index 9c439513c7..3d45508bea 100644 --- a/c/src/exec/posix/src/mqueuenametoid.c +++ b/c/src/exec/posix/src/mqueuenametoid.c @@ -32,7 +32,8 @@ * * _POSIX_Message_queue_Name_to_id * - * XXX + * Look up the specified name and attempt to locate the id + * for the associated message queue. */ int _POSIX_Message_queue_Name_to_id( @@ -48,7 +49,7 @@ int _POSIX_Message_queue_Name_to_id( if ( !name[0] ) return EINVAL; - if( strlen(name) > PATH_MAX ) /* XXX - Is strlen ok to use here ? */ + if( strlen(name) > PATH_MAX ) return ENAMETOOLONG; status = _Objects_Name_to_id( diff --git a/c/src/exec/posix/src/mqueueopen.c b/c/src/exec/posix/src/mqueueopen.c index 28ad7aab6b..c0ae35e465 100644 --- a/c/src/exec/posix/src/mqueueopen.c +++ b/c/src/exec/posix/src/mqueueopen.c @@ -51,7 +51,6 @@ mqd_t mq_open( _Thread_Disable_dispatch(); - /* XXX is attr used with out being initialized otherwise? */ if ( oflag & O_CREAT ) { va_start(arg, oflag); mode = (mode_t) va_arg( arg, mode_t ); @@ -104,8 +103,7 @@ mqd_t mq_open( } - /* XXX verify this comment... - * + /* * At this point, the message queue does not exist and everything has been * checked. We should go ahead and create a message queue. */ @@ -130,3 +128,7 @@ mqd_t mq_open( return (mqd_t) the_mq->Object.id; } + + + + diff --git a/c/src/exec/posix/src/semaphorecreatesupp.c b/c/src/exec/posix/src/semaphorecreatesupp.c index 8177cf9e94..448a658bab 100644 --- a/c/src/exec/posix/src/semaphorecreatesupp.c +++ b/c/src/exec/posix/src/semaphorecreatesupp.c @@ -44,17 +44,10 @@ int _POSIX_Semaphore_Create_support( } if ( name ) { - - if( strlen(name) > PATH_MAX ) { /* XXX - Is strlen ok to use here ? */ + if( strlen(name) > PATH_MAX ) { _Thread_Enable_dispatch(); set_errno_and_return_minus_one( ENAMETOOLONG ); } - - /* - * XXX Greater than NAME_MAX while POSIX_NO_TRUNC in effect. - * XXX Error description in POSIX book different for mq_open and mq_unlink - * Why??? - */ } the_semaphore = _POSIX_Semaphore_Allocate(); -- cgit v1.2.3