On Wed, 7 Sep 2016 09:12:55 +0200 Markus Heiser <markus.heiser@xxxxxxxxxxx> wrote: > according to your remarks I fixed the first and second patch. The third patch is > resend unchanged; OK, I've applied the first two, finally. > > Am 06.09.2016 um 14:28 schrieb Jonathan Corbet <corbet@xxxxxxx>: > > > > As others have pointed out, we generally want to hide the difference > > between functions and macros, so this is probably one change we don't > > want. > > I read "probably", so there might be a chance to persuade you ;) > > I'm not a friend of *information hiding* and since the index is sorted > alphabetical it does no matter if the entry is 'FOO (C function)' or 'FOO (C > macro)'. The last one has the right information e.g. for someone how is looking > for a macro. FOO is a function-like macro and not a function, if the author > describes the macro he might use the word "macro FOO" but in the index it is > tagged as C function. Information hiding is the only way we can maintain the kernel and stay sane. I have a hard time imagining why somebody would be looking for a macro in particular; the whole idea is that they really shouldn't have to care. So my inclination is to leave this one out, sorry. Thanks, jon -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
