[PATCH] Incidental setup.exe patches #3: Simplify packagedb task handling
Dave Korn
dave.korn.cygwin@googlemail.com
Wed Apr 14 16:59:00 GMT 2010
On 14/04/2010 15:41, Dave Korn wrote:
> I'll rewrite the comments now I know exactly what's going on and resend it
> later.
Here:
* root.cc (RootPage::OnNext): Don't construct a packagedb here nor
do deferred initialisation of static packagedb::task.
* source.cc (save_dialog): Don't construct a packagedb here, and
set static packagedb::task directly instead of chosen_db_task.
* package_meta.cc (packagemeta::action_caption): Don't bother to
construct a packagedb here, just access packagedb::task directly.
* package_db.cc: Move 'static members' comment near static members.
(chosen_db_task): Delete.
* package_db.h (chosen_db_task): Don't declare extern.
(packagedb): Extend comments on class.
OK now?
cheers,
DaveK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: setup-simplify-packagedb-task-handling-take2.diff
Type: text/x-c
Size: 4464 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-apps/attachments/20100414/7a3c1842/attachment.bin>
More information about the Cygwin-apps
mailing list