Karthik Nayak <[email protected]> writes: > Implement %(if:equals=<string>) wherein the if condition is only > satisfied if the value obtained between the %(if:...) and %(then) atom > is the same as the given '<string>'. > > Similarly, implement (if:notequals=<string>) wherein the if condition > is only satisfied if the value obtained between the %(if:...) and > %(then) atom is differnt from the given '<string>'. > > Add tests and Documentation for the same. > > Mentored-by: Christian Couder <[email protected]> > Mentored-by: Matthieu Moy <[email protected]> > Signed-off-by: Karthik Nayak <[email protected]> > ---
The fast that the patch touches only the narrow parts that are specific to %(if),%(then) and %(else) and does not have to touch any generic part (other than the populate_value() parser for obvious reasons) is a good signal that tells us that the basic structure of the code is very sound. I very much like the direction in which this series is going ;-) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
