Code review comment for lp:~canonical-dx-team/unity/unity.jason-fixes

Revision history for this message
Neil J. Patel (njpatel) wrote :

Looks good, but please send a GError into g_app_info_launch and g_warning the contents if it's set (and don't forget to free it too), We should be always settings GErrors and reporting them so we get the best info if Unity is breaking for someone.

review: Approve

« Back to merge proposal