UDSEntry compression ideas

David Faure faure at kde.org
Sun Sep 22 09:36:44 BST 2013


On Saturday 21 September 2013 15:45:55 Mark wrote:
> The patch: http://paste.kde.org/p2ece1fd8/
> Besides the line you said i also changed the line above. Hence asking for a
> "OK" to commit.

I said "Same for the other tests for d_type, they should all use == then."
so we agree :-)

> Then a few questions for frameworks. How do i merge this change back in
> frameworks?

s/back/forward/ :)
Don't.
I regularly merge 4.11 to master and master to frameworks.

> Note: i'm right now making a unittest for this specific case in frameworks,
> not for 4.11. Just a general "udsentrytest" which should later be extended
> to cover more of UDSEntry.

OK, great. Although listing is rather part of jobtest or of your new 
listdirtest... This wasn't a UDSEntry bug, but a KIO::listDir bug (as far as 
the user of the API is concerned).

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5





More information about the kfm-devel mailing list