On Sun, Jun 8, 2008 at 3:51 AM, Jameson Chema Quinn <[EMAIL PROTECTED]> wrote: > Sorry, Marco, but here is yet another version of the patch. I decided to > refactor the Config class out of bundlebuilder - all it was carrying in the > common start() case was bundle_name; and as for other cases, it is simpler > to call Builder(...params...) than Builder(Config(...params...)).
I'm using a config object because it's easier to carry around in the other Xer. I don't think having to create the Config is a big deal, calling it directly is not the primary use case anyway. But i you *really* feel that we need this bit of convenience, we can add a create_builder() which setups the config for you. Marco _______________________________________________ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
