+ Comment cleanup

4 files changed, 12 insertions, 15 deletions

diff --git a/cpukit/posix/src/mqueuecreatesupp.c b/cpukit/posix/src/mqueuecreatesupp.c
index ceebb2adc2..148f5b54cb 100644
--- a/cpukit/posix/src/mqueuecreatesupp.c
+++ b/cpukit/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/cpukit/posix/src/mqueuenametoid.c b/cpukit/posix/src/mqueuenametoid.c
index 9c439513c7..3d45508bea 100644
--- a/cpukit/posix/src/mqueuenametoid.c
+++ b/cpukit/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/cpukit/posix/src/mqueueopen.c b/cpukit/posix/src/mqueueopen.c
index 28ad7aab6b..c0ae35e465 100644
--- a/cpukit/posix/src/mqueueopen.c
+++ b/cpukit/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/cpukit/posix/src/semaphorecreatesupp.c b/cpukit/posix/src/semaphorecreatesupp.c
index 8177cf9e94..448a658bab 100644
--- a/cpukit/posix/src/semaphorecreatesupp.c
+++ b/cpukit/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();