{"push_id":828181615,"size":23,"distinct_size":5,"ref":"refs/heads/unify-access-contexts","head":"32d1cc988bf8ed1344f454d0a13ade2b02051e12","before":"298c33adce7162f38b0eb6e39c7cb6634daf4c78","commits":[{"sha":"8a71ddabd3616a12a1a21c1f715c8467101b62b7","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"Utility: enhance asString usage\n\nasString was not returning it's result, leading to more\ncomplicated usage.\n\nReturn the formated string instead of using an output parameter.\n\nSigned-off-by: Kevin Rocard <kevin.rocard@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/8a71ddabd3616a12a1a21c1f715c8467101b62b7"},{"sha":"c18f65626f3d5353b68d509f2fe651a7dd289ea1","author":{"email":"3a927494dd8fd4dfe8fa256992840dc4ba515be1@intel.com","name":"Patrick Benavoli"},"message":"Wrong XML \"Name\" attribute in enum value pairs\n\nWhen exporting an element's structure (via the C++ API or via the tuning\ninterface), a spurious \"Name\" attribute appears for EnumPair nodes, while\nonly \"Literal\" and \"Numerical\" attributes should be present. This is because\nthe XML import / export feature assummes the existence of a Name attribute\nfor all XML nodes. The Name default processing can however be skipped from the\ntoXml() methods by calling the base class' childrenToXml() method instead\nof the toXml() one.\n\nSigned-off-by: Patrick Benavoli <patrick.benavoli@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/c18f65626f3d5353b68d509f2fe651a7dd289ea1"},{"sha":"a74d574acf82466d9827381fd2e549f60ebdac31","author":{"email":"4d22a156b66fcbb389e15b35f814c350f6e644ba@gmail.com","name":"Miguel GAIO"},"message":"Remove specialization of convertTo<int16_t>()\n\nThis was a workaround of stlport issue.\nNo need anymore since stlport was dropped.\n\nSigned-off-by: Kevin Rocard <kevin.rocard@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/a74d574acf82466d9827381fd2e549f60ebdac31"},{"sha":"936e8f0f7025ae561673be010b9c3553ca90b79d","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"Remove a useless a wrong check in makeEncodable\n\nThis check was modified in a previous commit and was wrong since then. But it\nis also rather useless as it only serves as returning early in case the\nfunction is known to be a no-op.\n\nSigned-off-by: David Wagner <david.wagner@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/936e8f0f7025ae561673be010b9c3553ca90b79d"},{"sha":"7bb5debcacbf82e5c6c99673c16dd23e30fff6dd","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"Utility: Introduce unit test for Utility.h\n\nUtility functions were not tested,\nintroduce unit tests for all functions in Utility.h.\n\nSigned-off-by: Kevin Rocard <kevin.rocard@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/7bb5debcacbf82e5c6c99673c16dd23e30fff6dd"},{"sha":"975f5a0b3e9f5c36a0f643bbb1ca0d0b4f3c0bee","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"Utility: Simplify appendTitle implementation\n\nSigned-off-by: Kevin Rocard <kevin.rocard@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/975f5a0b3e9f5c36a0f643bbb1ca0d0b4f3c0bee"},{"sha":"e2db2773379ac066acaaa591c9780538853d0436","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"CommandHandle: Simplify help message implementation\n\nUse `std::string::string(size_t, char)` to simplify the\nconstruction of the help message.\n\nSigned-off-by: Kevin Rocard <kevin.rocard@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/e2db2773379ac066acaaa591c9780538853d0436"},{"sha":"9ffcd5430e65a705d252100ccc0c9d2cede2378f","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"CUtility should not be class, rather a namespace\n\nUtility.h contain functions that were all part of\na CUtility class. Nevertheless this class had no invariant\nnor asserting any invariant.\n\nThus this class was only used for its namespace aspect.\n\nUse a real namespace instead.\n\nSigned-off-by: Kevin Rocard <kevin.rocard@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/9ffcd5430e65a705d252100ccc0c9d2cede2378f"},{"sha":"7995895c0225018224fd2a164fd2fe7cdcdc6fa4","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"Merge pull request #279 from dawagner/fix-enumparam-name-attribute\n\nWrong XML \"Name\" attribute in enum value pairs","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/7995895c0225018224fd2a164fd2fe7cdcdc6fa4"},{"sha":"f2c85870d57a1ddad21ac5a2f4fa825cd1a22c66","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"Remove useless utils static methods in SubsystemObject\n\n- asInteger can be replaced with convertTo\n- asString can be replaced with std::to_string\n\nSigned-off-by: David Wagner <david.wagner@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/f2c85870d57a1ddad21ac5a2f4fa825cd1a22c66"},{"sha":"53e9c9ca32fb6c8ce916d7f9bd8964352a46c2ec","author":{"email":"3a927494dd8fd4dfe8fa256992840dc4ba515be1@intel.com","name":"Patrick Benavoli"},"message":"Define special case for convertTo<uint8_t>() and convertTo<int8_t>()\n\nThe generic version was converting uint8 as it was a character\n(uint8_t is an alias to unsigned char on most compiler).\nThus converting \"1\" would return 49 ie '1'.\n\nAs convertTo is thought as an _numerical_ conversion tool\n(contrary to boost::lexical_cast for example),\nforbid considering the input as a character and consider uint8_t\n(aka unsigned char) as a number exclusively\n\nSame for int8_t.\n\nSigned-off-by: Kevin Rocard <kevin.rocard@intel.com>\nSigned-off-by: David Wagner <david.wagner@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/53e9c9ca32fb6c8ce916d7f9bd8964352a46c2ec"},{"sha":"ef428c3e37af961473c50b09e981d4d8fde7a77b","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"convertTo: fire a static_assert when a convertion is forbidden\n\nThe error message is clearer than when triggering a generic compilation error.\n\nSigned-off-by: David Wagner <david.wagner@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/ef428c3e37af961473c50b09e981d4d8fde7a77b"},{"sha":"1413f7abd1e2a2151b78bca3805cc001ecf62b3c","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"Merge pull request #275 from krocard/utility-improvements\n\nUtility improvements:\r\n - asString now return the formated string leading to usage improvement\r\n - Fully unit test Utility.h\r\n - Simplify appendTitle implementation\r\n - CommandHandler: simplify the message formating","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/1413f7abd1e2a2151b78bca3805cc001ecf62b3c"},{"sha":"8126cf69cdb8eb4e75ffa143dec1241c0d664d50","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"Merge pull request #280 from dawagner/convert-to-bytes\n\nconvertTo enhancements:\r\n - correctly handle uint8_t and int8_t (deserialize as numerics, not character)\r\n - display a nicer error message if conversion is not supported (static_assert)\r\n - remove workaround stlport bug, as it is no longer supported","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/8126cf69cdb8eb4e75ffa143dec1241c0d664d50"},{"sha":"7768589eef4328c6f19f1238da714a48a588fd5b","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"Merge pull request #281 from dawagner/fix-makeEncodable\n\nRemove a useless and wrong check in makeEncodable","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/7768589eef4328c6f19f1238da714a48a588fd5b"},{"sha":"55296751b8f08b06c8c4c3f7d92c345e4a96457c","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"Merge pull request #282 from dawagner/remove-some-useless-util-methods\n\nRemove useless utils static methods in SubsystemObject","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/55296751b8f08b06c8c4c3f7d92c345e4a96457c"},{"sha":"c1f47d636cfe3e6c91b836fbe53fb3fc5e20744f","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"Remove utility/test from the coverage\n\nSince we want the coverage report to show only the coverage of code that will\nrun in production, we don't want tests to artificially increase the coverage.\n\nSigned-off-by: David Wagner <david.wagner@intel.com>","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/c1f47d636cfe3e6c91b836fbe53fb3fc5e20744f"},{"sha":"7bedd3086579477e9cd1c4729abaf0f42827a89f","author":{"email":"f6672397850e914591889d29dcaf9077eb1a8a4f@intel.com","name":"Kevin Rocard"},"message":"Merge pull request #283 from dawagner/remove-utility-test-from-coverage\n\nExclude utility/test from the coverage","distinct":false,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/7bedd3086579477e9cd1c4729abaf0f42827a89f"},{"sha":"f62231c6aaa75de75edc243e724f6224c9608351","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"'dump' code path: return by value instead of using output parameters\n\nIt makes the code more readable and obvious.\n\nSigned-off-by: David Wagner <david.wagner@intel.com>","distinct":true,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/f62231c6aaa75de75edc243e724f6224c9608351"},{"sha":"15bf1f6200196fdeec64d7c189ce4171112f073b","author":{"email":"60228a510f9a76940848fa2d29321fc4edfc2119@intel.com","name":"David Wagner"},"message":"Overrive logValue() for Configurable Elements\n\nConfigurableElement and its derived classes were all casting an access\ncontext. Since logValue() is recursively called on children (which, in the case\nof ConfigurableElement are also derived classes of ConfigurableElement), we can\ndo the static_cast once for all children.\n\nSigned-off-by: David Wagner <david.wagner@intel.com>","distinct":true,"url":"https://api.github.com/repos/dawagner/parameter-framework/commits/15bf1f6200196fdeec64d7c189ce4171112f073b"}]}