Review: Approve I only found two cosmetic issues: o In test applications argc is not checked, but as these are just for testing it's ok. o tests/test-json-client.c argc and argv have switched places
But ready for merge in my opinion. -- https://code.launchpad.net/~ted/dbusmenu/json-it-out/+merge/28696 Your team ayatana-commits is subscribed to branch lp:dbusmenu. _______________________________________________ Mailing list: https://launchpad.net/~ayatana-commits Post to : [email protected] Unsubscribe : https://launchpad.net/~ayatana-commits More help : https://help.launchpad.net/ListHelp
