[Panel-devel] [PATCH] Package and PackageStructure cleanup

Paolo Capriotti p.capriotti at gmail.com
Sun Nov 25 01:01:52 CET 2007


Hi,
while working on background selection with ruphy, I've found some 
oddities in the existing Package code. Here is a cleanup patch that:
1) replaces all const char* with QString in the API;
2) changes the Package constructor to a single path argument;
3) makes some methods in PackageStructure const;
4) updates the only usage of Package I could find, and unit tests.
Ok to commit?

While we are at it, I was wondering if a more fundamental refactoring of 
the Package classes could be useful. Since there is almost no code using 
them, I would like to know what the use cases are, so that we can think 
about improving the API and functionality in the next few days.

Paolo Capriotti
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cleanup2.patch
Type: text/x-diff
Size: 19377 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071125/3f7428f2/attachment-0001.bin 


More information about the Panel-devel mailing list