On Thu, Nov 17, 2016 at 7:13 PM, Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> wrote: > > Not much important to me, but said above, it's confusing: two args, > one called 'f', the other one 'fn' and one of them is unused ... Points taken. > >> Or maybe change it to "node" instead of "f", will that be better to understand? > Not much I think, 'node' has even less meaning to me than 'f'or 'fn'. > Better then to focus on the (potential) purpose of this arg. > If it's for the position, maybe pass only sym->pos. OK. I will just apply the patch. We can always add the argument back if there is a real need for it. Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
