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 not convinced that's a good idea. But anyway, let's not increase the scope of the patch, as I told you I already had problems to review it in his current form. I'll review the old versions of the patch and when that's done we can consider this change. Thanks, Marco _______________________________________________ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
