On Wed, 13 Jun 2007, Joerg Sonnenberger wrote:

On Wed, Jun 13, 2007 at 08:26:38AM -0700, Garrett Cooper wrote:
Sorry -- actually I meant that (along similar lines), there was a
program with the following lines:

vsystem("/bin/chmod +x %s", filename);

and I replaced it with:

chmod(filename, (mode_t) ( S_IXUSR | S_IXGRP | S_IXOTH ));

You supposedly mean stat + chmod here, right? Trivial errors like this
are easy to make.

Yes ><. Good catch, thanks :).

Next step, eliminating the linked list structure in favor or red-black
trees, maybe.

Due to the way pkg_install works, this most likely is just adding
complexity for no gain, it might actually slow it down.

Hmmm... the only thing is that it does the linked list traversal a number of 
times per dependency. I'll take a look in gdb at the size of each dependency 
and then confer with Kirill (my mentor) about that a bit more.

-Garrett

_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to