[rekonq] Re: Review Request: A bit of cleanup in Application.
Pierre Rossi
pierre.rossi at gmail.com
Sat Jan 8 22:51:38 CET 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100316/
-----------------------------------------------------------
(Updated Jan. 8, 2011, 9:51 p.m.)
Review request for rekonq and Benjamin Poulain.
Changes
-------
Sure, QScopedPointer is so nice, it'd be a waste not to use it. :)
Summary
-------
I believe we don't need static members in QWeakPointers for all the *Managers, static getter functions would do the job.
This addresses bug N/A.
/show_bug.cgi?id=N/A
Diffs (updated)
-----
src/application.h b30e337
src/application.cpp 466a0a4
Diff: http://git.reviewboard.kde.org/r/100316/diff
Testing
-------
"compiles and runs" ™
Thanks,
Pierre
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/rekonq/attachments/20110108/d4c444ec/attachment.htm
More information about the rekonq
mailing list