On 12/08/11 13:58, Brock Pytlik wrote:
Webrev:
https://cr.opensolaris.org/action/browse/pkg/bpytlik/19098-v1

Bugs:
19098 allow_action can be faster for both variant and facet
19100 imageplan uses apx as a module but doesn't import as module as apx

src/modules/facet.py:
  need copyright update

lines 134-136: did you check and see if f.startswith("facet.") was faster/slower than f[:6]? In many cases, startswith is faster because slicing requires the creation of a temporary object and additional memory allocation.

src/modules/variant.py:
  lines 83, 93: same as above in regards to startswith

Otherwise, very cool.

-Shawn

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to