diff options
author | Aaron Griffin <aaron@archlinux.org> | 2007-02-07 20:22:36 +0000 |
---|---|---|
committer | Aaron Griffin <aaron@archlinux.org> | 2007-02-07 20:22:36 +0000 |
commit | 7969082a7c75c5367eaf5d8f42b93abbe17a3b4b (patch) | |
tree | c3427a28ccb0e80fd48a39e2d37b8646c0c93b55 | |
parent | 01b3f1e0f5b962968edd3992b0eb5a0596964e1e (diff) | |
download | pacman-7969082a7c75c5367eaf5d8f42b93abbe17a3b4b.tar.xz |
*** empty log message ***
-rw-r--r-- | TODO.aaron | 8 |
1 files changed, 8 insertions, 0 deletions
@@ -22,6 +22,14 @@ - clear up list allocation/deallocation - some lists need to be free'd, some do not and there is no clear indication WHEN this should happen. - remove DB entries (directories) on a read error? + - Add a pm_errstr error string, which will allow us to do things like: + pm_errno = PM_ERR_LIBARCHIVE; + pm_errstr = archive_error_string(archive); + or: + pm_errno = PM_ERR_LIBDOWNLOAD; + pm_errstr = downloadLastErrString; + This way we don't break abstraction when returning specific errors in + cases like the above. * pacman: A LOT of functions are way too long. There should be an upper limit of 100-200 lines. _alpm_add_commit is around 600 lines, and is far too complex. |