{"push_id":636046829,"size":5,"distinct_size":0,"ref":"refs/heads/criterion_rework","head":"5be27ed053a0dce711b8014b170e013da2afeead","before":"1556a9a9680211f6a1934aa76bbba83e59674732","commits":[{"sha":"c200abf6f6d52188fbf104055c6bb7f5c2b54467","author":{"email":"2deac8eb6d4468174a95d7398a201f3232180ec0@intel.com","name":"Jules Clero"},"message":"Make a separate library for Criteria\n\nCriteria interface has been enough reworked to introduce a separated\nlibrary. This library exposes a client interface to let the client dealt\nwith criteria and larger interface through Criteria class which allows\nthe parameter-framework to handle criteria.\n\nSigned-off-by: Jules Clero <julesx.clero@intel.com>","distinct":false,"url":"https://api.github.com/repos/clero/parameter-framework/commits/c200abf6f6d52188fbf104055c6bb7f5c2b54467"},{"sha":"e0d6af09da0b934406423a0839bf246b5cc55ee5","author":{"email":"2deac8eb6d4468174a95d7398a201f3232180ec0@intel.com","name":"Jules Clero"},"message":"Fix default state display for InclusiveCriterion\n\nInclusiveCriterion has a default state called \"none\", this state was\ndisplayed through default Criterion behavior which is: if no valid state\nis available, let's display \"<none>\".\nThis patch test explicitly the default value when formatting the state\nof an InclusiveCriterion. It also introduces the test getLiteralValue\nreturn when formatting a standard Criterion rather than testing the\ngenerated string emptiness.\nA difference can now be made between an inclusive criterion default\nvalue and an exclusive criterion with an unknown state.\n\nSigned-off-by: Jules Clero <julesx.clero@intel.com>","distinct":false,"url":"https://api.github.com/repos/clero/parameter-framework/commits/e0d6af09da0b934406423a0839bf246b5cc55ee5"},{"sha":"36b9f69aa0267a8f4582f00ff42a698f74f9a86e","author":{"email":"2deac8eb6d4468174a95d7398a201f3232180ec0@intel.com","name":"Jules Clero"},"message":"Update Criterion type display string for consistency\n\nCriterion type, which can be inclusive or exclusive is displayed\nsometimes with a capital letter and sometimes not.\nThis patch corrects this behaviour to always have the first letter\nin capital.\n\nSigned-off-by: Jules Clero <julesx.clero@intel.com>","distinct":false,"url":"https://api.github.com/repos/clero/parameter-framework/commits/36b9f69aa0267a8f4582f00ff42a698f74f9a86e"},{"sha":"04e35c2fef6f7e1c5d62dc30c71b1ca2b00fe1eb","author":{"email":"2deac8eb6d4468174a95d7398a201f3232180ec0@intel.com","name":"Jules Clero"},"message":"Add unit test to criterion library\n\nAs the criterion is now an isolated library, unit tests can written.\nThis patch covers the entire criterion library.\n\nSigned-off-by: Jules Clero <julesx.clero@intel.com>","distinct":false,"url":"https://api.github.com/repos/clero/parameter-framework/commits/04e35c2fef6f7e1c5d62dc30c71b1ca2b00fe1eb"},{"sha":"5be27ed053a0dce711b8014b170e013da2afeead","author":{"email":"2deac8eb6d4468174a95d7398a201f3232180ec0@intel.com","name":"Jules Clero"},"message":"Rework Inclusive Criterion numerical value definition\n\nPreviously, numerical values of inclusive and standard (exclusive)\ncriterion was not handled in the same way.\nExclusive criterion can have any integer as numerical value. On\nthe contrary, inclusive criterion can only have a numerical state\nwhich is a power of 2. It comes from the fact that a bit mask is\ninternally used to handle inclusive criterion state.\nThis implementation forces the client to register criteria with\na mask where only one bit is set.\nFor example, to register a literal 'A' in the third position of\na bit mask we have to do criterion.addValuePair('A', 1 << 3).\nThis is an error prone interface because it is not consistent\nwith the exclusive criterion interface.\nThis patch rework this interface to register the literal value\nwith the desired bit mask index (e.g: criterion.addValuePair('A', 3)).\n\nSigned-off-by: Jules Clero <julesx.clero@intel.com>","distinct":false,"url":"https://api.github.com/repos/clero/parameter-framework/commits/5be27ed053a0dce711b8014b170e013da2afeead"}]}