Messages in this thread | | | Date | Sat, 26 Mar 2022 11:24:15 +0100 | From | Mauro Carvalho Chehab <> | Subject | Re: [PATCH v2 1/4] scripts/get_feat.pl: allow output the parsed file names |
| |
Em Fri, 25 Mar 2022 18:02:09 -0600 Jonathan Corbet <corbet@lwn.net> escreveu:
> Mauro Carvalho Chehab <mchehab@kernel.org> writes: > > > Btw, maybe we could replace them tree with a Sphinx comment, like: > > > > get_feat.pl: > > .. FILE <file_name> > > kernel-doc: > > .. LINE <line_number> > > get_abi.pl: > > .. FILE_LINE <file_name>:<line_number> > > > > (or something similar) > > > > Just let me know what you prefer and I can take care of the needed > > changes on this patch and, if it is the case, writing the extra patches > > in order to use the same model on kernel-doc and get_abi.pl. > > If it were just me, I'd just put "FILE <name>" or something simple.
This won't be a problem here, but I would prefer to use something that would have zero chances of causing issues at kernel-doc and on other similar scripts that we might end needing. Using a valid ReST tag like starting it with ".." should fulfill such goal.
A side effect is that the output of the script would be a valid ReST file, either with or without file/line information, which sounds a good idea on my eyes.
> I don't really have a strong opinion on the matter though; it's not like > people have to actually look at these things.
Yes. That's why I ended using #define, the actual meta-tag won't matter for people.
> I was mostly curious as > to why you'd done it that way. I can take the original patches or any > of the variants above; just let me know which you like best and we'll > get this done.
Ok. I'll resubmit the patch changing to the comments tag. I'll add patches also for kernel-doc and get_abi.pl for all of them to use the same meta-tag.
Thanks, Mauro
| |