Shawn Walker wrote:
[snip]

server/catalog:
We could probably just remove the checks for .pag and .dir? Those haven't existed since the 2008.05 release.

1151: what are the possible classes that could be coming in here?

It's marked as a classmethod now so it can access __parse_entry.
Ok, I'll try again. Why is it a static method instead of a class method?

I'll assume you meant why is it a @classmethod instead of a @staticmethod since it is a @classmethod right now :)

In that case, yes, I could change it to that, but then I would have to call ServerCatalog.__parse_entry instead and I don't care for that form since it embeds the class name in the code.

Does this suggest that instead of using static method by default like we've been doing, we should be using classmethod much more? (Maybe we have been and I'm oblivious, but I don't remember seeing a classmethod decorator before.) It seems like maybe we should migrate that way.

Brock

Cheers,

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

Reply via email to