[pacman-dev] pacman -Q broken

Xavier shiningxc at gmail.com
Sat Jun 2 06:47:07 EDT 2007


Hello,

this commit broke pacman -Q :
http://projects.archlinux.org/git/gitweb.cgi?p=pacman.git;a=commit;h=722db4535ae6690d8834ffebf3a0de3a880188f9

If the list of targets is empty, then it doesn't enter the loop, and
no code is executed.
Handling the case where there are no targets inside the loop visiting
the targets is pretty funny in my opinion.
So I tried to handle this case separately, but I find the whole
function still very messy. Because this case is handled differently
depending on the type of query.
So what about making new function for each different query ? And not
trying to handle all the cases in the same big for loop, but having a
separate loop for each.
Maybe it could be better that way.




More information about the pacman-dev mailing list