svn commit: trunk/busybox: e2fsprogs/old_e2fsprogs/blkid libbb tes etc...
vda at busybox.net
vda at busybox.net
Mon Mar 17 09:05:22 UTC 2008
Author: vda
Date: 2008-03-17 02:05:21 -0700 (Mon, 17 Mar 2008)
New Revision: 21354
Log:
clean up TODO file. No real code changes.
Modified:
trunk/busybox/TODO
trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/cache.c
trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/dev.c
trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/tag.c
trunk/busybox/libbb/appletlib.c
trunk/busybox/libbb/dump.c
trunk/busybox/testsuite/all_sourcecode.tests
Changeset:
Modified: trunk/busybox/TODO
===================================================================
--- trunk/busybox/TODO 2008-03-17 09:04:04 UTC (rev 21353)
+++ trunk/busybox/TODO 2008-03-17 09:05:21 UTC (rev 21354)
@@ -7,19 +7,16 @@
between your work and theirs. But otherwise, all of these are fair game.
Rob Landley <rob at landley.net>:
- Add BB_NOMMU to platform.h and migrate __uClinux__ tests to that.
- #if defined __UCLIBC__ && !defined __ARCH_USE_MMU__
Add a libbb/platform.c
Implement fdprintf() for platforms that haven't got one.
Implement bb_realpath() that can handle NULL on non-glibc.
Cleanup bb_asprintf()
- Migrate calloc() and bb_calloc() occurrences to bb_xzalloc().
Remove obsolete _() wrapper crud for internationalization we don't do.
Figure out where we need utf8 support, and add it.
sh
- The command shell situation is a big mess. We have three or four different
+ The command shell situation is a big mess. We have three different
shells that don't really share any code, and the "standalone shell" doesn't
work all that well (especially not in a chroot environment), due to apps not
being reentrant.
@@ -124,9 +121,6 @@
As yet unclaimed:
----
-find
- doesn't understand (), lots of susv3 stuff.
-----
diff
Make sure we handle empty files properly:
From the patch man page:
@@ -143,10 +137,6 @@
And while we're at it, a new patch filename quoting format is apparently
coming soon: http://marc.theaimsgroup.com/?l=git&m=112927316408690&w=2
---
-ps / top
- Add support for both RSS and VSIZE rather than just one or the other.
- Or make it a build option.
----
man
It would be nice to have a man command. Not one that handles troff or
anything, just one that can handle preformatted ascii man pages, possibly
@@ -310,9 +300,6 @@
Replace deprecated functions.
-bzero() -> memset()
---
-sigblock(), siggetmask(), sigsetmask(), sigmask() -> sigprocmask et al
----
vdprintf() -> similar sized functionality
---
Modified: trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/cache.c
===================================================================
--- trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/cache.c 2008-03-17 09:04:04 UTC (rev 21353)
+++ trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/cache.c 2008-03-17 09:05:21 UTC (rev 21354)
@@ -35,8 +35,7 @@
DBG(DEBUG_CACHE, printf("creating blkid cache (using %s)\n",
filename ? filename : "default cache"));
- if (!(cache = (blkid_cache) calloc(1, sizeof(struct blkid_struct_cache))))
- return -BLKID_ERR_MEM;
+ cache = xzalloc(sizeof(struct blkid_struct_cache));
INIT_LIST_HEAD(&cache->bic_devs);
INIT_LIST_HEAD(&cache->bic_tags);
Modified: trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/dev.c
===================================================================
--- trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/dev.c 2008-03-17 09:04:04 UTC (rev 21353)
+++ trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/dev.c 2008-03-17 09:05:21 UTC (rev 21354)
@@ -20,8 +20,7 @@
{
blkid_dev dev;
- if (!(dev = (blkid_dev) calloc(1, sizeof(struct blkid_struct_dev))))
- return NULL;
+ dev = xzalloc(sizeof(struct blkid_struct_dev));
INIT_LIST_HEAD(&dev->bid_devs);
INIT_LIST_HEAD(&dev->bid_tags);
Modified: trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/tag.c
===================================================================
--- trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/tag.c 2008-03-17 09:04:04 UTC (rev 21353)
+++ trunk/busybox/e2fsprogs/old_e2fsprogs/blkid/tag.c 2008-03-17 09:05:21 UTC (rev 21354)
@@ -21,8 +21,7 @@
{
blkid_tag tag;
- if (!(tag = (blkid_tag) calloc(1, sizeof(struct blkid_struct_tag))))
- return NULL;
+ tag = xzalloc(sizeof(struct blkid_struct_tag));
INIT_LIST_HEAD(&tag->bit_tags);
INIT_LIST_HEAD(&tag->bit_names);
Modified: trunk/busybox/libbb/appletlib.c
===================================================================
--- trunk/busybox/libbb/appletlib.c 2008-03-17 09:04:04 UTC (rev 21353)
+++ trunk/busybox/libbb/appletlib.c 2008-03-17 09:05:21 UTC (rev 21354)
@@ -264,6 +264,7 @@
s = buffer;
if (!fgets(s, sizeof(buffer), f)) { /* Are we done? */
+// why?
if (ferror(f)) { /* Make sure it wasn't a read error. */
parse_error("reading");
}
Modified: trunk/busybox/libbb/dump.c
===================================================================
--- trunk/busybox/libbb/dump.c 2008-03-17 09:04:04 UTC (rev 21353)
+++ trunk/busybox/libbb/dump.c 2008-03-17 09:05:21 UTC (rev 21354)
@@ -95,7 +95,7 @@
*/
for (nconv = 0, fmtp = fu->fmt; *fmtp; nextpr = &pr->nextpr) {
/* NOSTRICT */
- /* DBU:[dvae at cray.com] calloc so that forward ptrs start out NULL*/
+ /* DBU:[dvae at cray.com] zalloc so that forward ptrs start out NULL*/
pr = xzalloc(sizeof(PR));
if (!fu->nextpr)
fu->nextpr = pr;
@@ -704,7 +704,7 @@
/* allocate a new format unit and link it in */
/* NOSTRICT */
- /* DBU:[dave at cray.com] calloc so that forward pointers start out NULL */
+ /* DBU:[dave at cray.com] zalloc so that forward pointers start out NULL */
tfu = xzalloc(sizeof(FU));
*nextfu = tfu;
nextfu = &tfu->nextfu;
Modified: trunk/busybox/testsuite/all_sourcecode.tests
===================================================================
--- trunk/busybox/testsuite/all_sourcecode.tests 2008-03-17 09:04:04 UTC (rev 21353)
+++ trunk/busybox/testsuite/all_sourcecode.tests 2008-03-17 09:05:21 UTC (rev 21354)
@@ -73,7 +73,7 @@
# don't allow obsolete functions
#
find $srcdir/.. '(' -name '*.c' -o -name '*.h' ')' -print0 | xargs -0 \
- grep -E -e '\<(bcmp|bcopy|bzero|getwd|index|mktemp|rindex|utimes)\>[[:space:]]*\(' \
+ grep -E -e '\<(bcmp|bcopy|bzero|getwd|index|mktemp|rindex|utimes|sigblock|siggetmask|sigsetmask)\>[[:space:]]*\(' \
| sed -e "s:^$srcdir/\.\./::g" > src.obsolete.funcs
testing "Obsolete function usage" "cat src.obsolete.funcs" "" "" ""
rm -f src.obsolete.funcs
More information about the busybox-cvs
mailing list