More patches

[BogDan]

Hi,

Commits 1 to 6 pushed !


I need your help to understand source mapping mechanism, now it seem you can't set its values from within sdk installer because it uses local users settings and AFAIK the SDK installer can't set those settings, the problem is we need (and use) those settings with Necessitas SDK, we ship the libs compile also with debug information, so, the user only needs to install qt sources and it can step into qt sources, it help a lot us because qt for android is not very stable. Should I/you rewrite that part ?


Thank you !


Cheers,
BogDan.


>________________________________
>From: Daniel Teske <daniel.teske at nokia.com>
>To: necessitas-devel at kde.org
>Sent: Thursday, September 8, 2011 9:41 PM
>Subject: More patches
>
>Hi,
>
>another round of small patches
>
>0001:
>  That patch removes a warning for me. I'm a little bit unsure where that
>  zorder line comes from, or what the effect is, but removing the lines didn't
>  seem to have any averse effects and it removed the warning for me
>
>0002:
>  The member is simply not used anywhere, thus I think it can be removed
>  entirely
>
>0003:
>  More coding style stuff that I missed in the first pass. I fixed a few overly
>  long lines, though there are many lines that are rather long....
>
>0004:
>  This patch adjusts the code to a change in Creator which removed the init()
>  function.
>
>0005:
>  This model seems to be unused. I didn't check the history to figure out why
>  it exists, or when that happened. If there's any reason for keeping it,
>  leave the patch out.
>
>0006:
>   I got a QProcess: Destroyed while still running if I started a application
>   on the device without closing the outputpane in between, this patch fixes
>   that warning.
>
>0007:
>   The AndroidRunConfiguration returns 0 for the configuration widget, which
>   lead to a warning: "Cannot add null widget to layout". I think it's okay to 
>   return 0 from that function, and thus opted to patch the 
>   RunSettingsPropertiesPage. I'll submit the patch to master, thus there's
>   no need to apply it to the android branch.
>
>daniel
>
>_______________________________________________
>Necessitas-devel mailing list
>Necessitas-devel at kde.org
>https://mail.kde.org/mailman/listinfo/necessitas-devel
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/necessitas-devel/attachments/20110909/ddfd1a9c/attachment.html>