[Merge] lp:~dandrader/qtubuntu/betterSessionName into lp:qtubuntu

Gerry Boland gerry.boland at canonical.com
Wed Jun 1 15:58:06 UTC 2016


> > Do you really need to read the FileInfo object? Why not just drop the ".qml"
> from the file name?
> Yes. The argument could be a full file path like this:
> 
> /foo/bar/whatever/zumba.qml
> 
> Which would make for a cumbersome session name. QFileInfo neatly and
> safely parses that filepath for me.
> 
> I like the ".qml" suffix as it tells us it's a plain qml app run via
> qmlscene.

Yeah, that's ok. I just want to keep app startup light, using QFileInfo to parse a file name just strikes me as heavy for something that only those reading file logs will appreciate. But I won't block on that.


> > + u_application_id_new_from_stringn(sessionName.data(),
> sessionName.count());
> > Does the api expect sessionName to always exist, or will it make a copy
> itself?
> 
> It makes a copy. What a horrible API it would be otherwise.

You never know :)
-- 
https://code.launchpad.net/~dandrader/qtubuntu/betterSessionName/+merge/296198
Your team Ubuntu Phablet Team is subscribed to branch lp:qtubuntu.



More information about the Ubuntu-reviews mailing list