Changeset 6921178 in mainline

Timestamp:

2011-08-17T20:25:05Z (13 years ago)

Author:

Petr Koupy <petr.koupy@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

c53a705

Parents:

55b1efd

Message:

Edited or removed some TODO/FIXME comments.

Location:

uspace/lib/posix

Files:

• ctype.c (modified) (1 diff)
• errno.h (modified) (1 diff)
• fnmatch.c (modified) (1 diff)
• stdbool.h (modified) (1 diff)
• stdio/scanf.c (modified) (2 diffs)
• string.h (modified) (1 diff)
• strings.h (modified) (1 diff)

uspace/lib/posix/ctype.c

@@ -37 +37 @@
 
 #include "ctype.h"
-
-// TODO: propose for inclusion in libc
 
 /**

uspace/lib/posix/errno.h

@@ -57 +57 @@
  * redefinition for such error codes.
  *
- * FIXME: maybe all HOS error codes should be redefined
+ * XXX: maybe all HOS error codes should be redefined
  *
  * NOTE: This redefinition is slightly POSIX incompatible, since the

uspace/lib/posix/fnmatch.c

@@ -624 +624 @@
 }
 
-// FIXME: put the testcases somewhere else
+// FIXME: put the testcases to the app/tester after fnmatch is included into libc
 
 #if 0

uspace/lib/posix/stdbool.h

@@ -36 +36 @@
 #define POSIX_STDBOOL_H_
 
-// TODO: propose for inclusion in libc and drop bool.h,
-//       it's a very silly incompatibility with standard C
-
 #ifdef LIBC_BOOL_H_
         #error "You can't include bool.h and stdbool.h at the same time."

uspace/lib/posix/stdio/scanf.c

@@ -707 +707 @@
                                                         break;
                                                 case LMOD_t: ;
-                                                        // FIXME: What is unsigned counterpart of the ptrdiff_t?
+                                                        // XXX: What is unsigned counterpart of the ptrdiff_t?
                                                         size_t *pt = va_arg(arg, size_t *);
                                                         *pt = (size_t) ures;
@@ -1221 +1221 @@
 }
 
-// FIXME: put the testcases somewhere else
+// FIXME: put the testcases to the app/tester after scanf is included into libc
 
 #if 0

uspace/lib/posix/string.h

@@ -51 +51 @@
  * void *memmove(void *, const void *, size_t);
  *
- * unimplemented due to missing locales
+ * TODO: not implemented due to missing locale support
  *
  * int      strcoll_l(const char *, const char *, locale_t);
  * char    *strerror_l(int, locale_t);
  * size_t   strxfrm_l(char *restrict, const char *restrict, size_t, locale_t);
- *
  */
-
-// TODO: provide *_l once there is locale.h
 
 #ifndef NULL

uspace/lib/posix/strings.h

@@ -48 +48 @@
 #endif
 
-/* TODO: not implemented due to missing locale.h
+/* TODO: not implemented due to missing locale support
  *
  * int strcasecmp_l(const char *, const char *, locale_t);