I'm a bit dumbfounded that this kind of Qt metaprogramming would be necessary 
for solving a problem as basic as what we have here. At the minimum I think 
this code would need to be commented, because if anyone stumbles upon this code 
in the future it'd be a big WTF moment without clarifying comments. Comments 
should explain what it fixes, why it had to be done this way.

-- 
https://code.launchpad.net/~renatofilho/ubuntu-filemanager-app/fix-network-crash/+merge/314870
Your team Ubuntu File Manager Developers is subscribed to branch 
lp:ubuntu-filemanager-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to     : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to