From: skeezix Date: Tue, 15 Dec 2009 16:05:48 +0000 (-0500) Subject: Add a couple things to TODO list so I don't forget (TODO.txt) X-Git-Tag: Release-2010-05/1~143^2~7 X-Git-Url: http://git.openpandora.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=954c1700b6f2cd7ba99244ce6df687db1671ce9f;p=pandora-libraries.git Add a couple things to TODO list so I don't forget (TODO.txt) Remove some inaccurate comments from pndnotifyd --- diff --git a/TODO.txt b/TODO.txt index 8d9c843..22f6f7d 100644 --- a/TODO.txt +++ b/TODO.txt @@ -1,14 +1,26 @@ -Some things to be done.. (see also the main wiki 'todo' page) +- pndnotifyd drops icons into a conf'd location (/tmp currently); it skips emitting them if that path is already + present (saves time after first emit!) .. the risk here is that the icon has changed or been removed, and we just + keep using the old one. Perhaps pndnotifyd should check the length of the icon (or a CRC32 or something), on occasion? -- UNICODE everywhere.. this will be a lot of work I'm sure. (replacing all the char* and related consumers) +- pndnotifyd doesn't delete icons; it leaves them in /tmp as a cache of sorts. /tmp may not be wisest if it gets turfed on + boot, but I'm up in the air on that one; we want caching, but not unlimited lifespan, so /tmp seems useful. (Unless /tmp + is in RAM, which it could be, should check.) ANYWAY, my point is.. + - should we conf how many icons/bytes can be consumed in 'cache' this way? + - more to point, should we remove old icons (ie: say, a week old) by default, so occasional re-caching occurs, but + we also clean up unused icons? We could also keep an access-count or somesuch. Just sayin' + +- fill in the blanks in pnd-logger + +- more useful PXML.xml parser, to cover all the fields and provide iterators, and better support the treelike nature - applist memleak; ie: all those disco_t apps returned are never free'd properly include/pnd_discovery.h:// TODO: A way to release the disco-lists and reclaim RAM :) -- put in code to fetch clock speed, and add clock speed handler to pnd_apps_exec +- put in code to fetch clock speed, and add clock speed handler to pnd_apps_exec? maybe unify the whole emit-dotdesktop and + exec routines o they dont' have two representations of command line args .. -- remove -u's from pnd_apps_exec and pnd_emit_dotdesktop +- remove -u's from pnd_apps_exec and pnd_emit_dotdesktop? - replace brute force memcmp() in pnd pnd with a fancy pants CS alg - make pnd_pndfiles seek of PXML more efficient; fread ( 1, nmemb ) gotta be slow :) diff --git a/apps/pndnotifyd.c b/apps/pndnotifyd.c index 31d9c92..0fa18ba 100644 --- a/apps/pndnotifyd.c +++ b/apps/pndnotifyd.c @@ -4,10 +4,6 @@ * */ -// TODO: Catch HUP and reparse config -// TODO: Should perhaps direct all printf's through a vsprintf handler to avoid redundant "if ! g_daemon_mode" -// TODO: During daemon mode, should perhaps syslog or log errors - #include // for stdio #include // for exit() #include // for exit()