[minion-cvs] Bugfixes for armoring, refactor to use readFile/WriteFi...

[nickm@seul.org (Nick Mathewson)]

Update of /home/minion/cvsroot/src/minion
In directory moria.mit.edu:/tmp/cvs-serv15318

Modified Files:
	TODO 
Log Message:
Bugfixes for armoring, refactor to use readFile/WriteFile wrappers.

Common:
- Add "tryUnlink" function to perform common unlink-this-file-if-you-can
  operation.
- Debug armorText and unarmorText.
- Document writeFile and readFile

Packet:
- Remove dead comment.

test: 
- Remove redundant readFile and writeFile
- Add tests for armorText and unarmorText.

ClientMain, Common, ServerInbox, ServerList, EventStats, HashLogs, Modules,
   ServerKeys, ServerMain, ServerQueue:
- Use writeFile, readFile, tryUnlink where possible.
- Avoid "except" without "raise" or sys.exc_info()
- Avoid "except OSError" without checking errno or displaying error.


Index: TODO
===================================================================
RCS file: /home/minion/cvsroot/src/minion/TODO,v
retrieving revision 1.116
retrieving revision 1.117
diff -u -d -r1.116 -r1.117
--- TODO	5 Jun 2003 02:27:29 -0000	1.116
+++ TODO	5 Jun 2003 05:24:23 -0000	1.117
@@ -168,7 +168,7 @@
         o Partially superceded descriptors should get replaced in dir,
           right?
         o Resolve all the memory leaks.
-        - Finish all documentation, resolve all XXXX004s
+        . Finish all documentation, resolve all XXXX004s
         - Add a warning banner.
         - Remaining unit tests
                 - Tests for remembering whether keys are published