Karthik Nayak <karthik.188@xxxxxxxxx> writes: >> This problem will go away if you introduce the 'valid_atom' field in >> the patch which actually needs it (as suggested above) rather than in >> this patch. > > Yup, agreed. > Thanks for your suggestions. In addition, most of the lines in this patch should become unnecessary even after you start using the "parser" field. As the majority of fields are compared as strings, and only a selected few fields need custom parsers, you do not have to explicitly write FIELD_STR everywhere, as long as you make sure the value of FIELD_STR is zero (and use parser==NULL as a sign that no custom parser is used, which I think you already do). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html