On Tue, 2007-02-13 at 11:35 -0800, Randy Dunlap wrote: > > Yup, that fixes it for structs. I guess I forgot to mention this earlier > > because I didn't notice it until now... the same bug exists for > > functions as well. > > updated patch. Thanks. Thanks, works great. > --- > > From: Randy Dunlap <[email protected]> > > Output of a function or struct in html mode needs to include the short > description from the function/struct name line in the output title line. > > Signed-off-by: Randy Dunlap <[email protected]> So if anyone cares: Acked-by: Johannes Berg <[email protected]> > --- > scripts/kernel-doc | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > --- linux-2.6.20-git7.orig/scripts/kernel-doc > +++ linux-2.6.20-git7/scripts/kernel-doc > @@ -452,7 +452,7 @@ sub output_struct_html(%) { > my %args = %{$_[0]}; > my ($parameter); > > - print "<h2>".$args{'type'}." ".$args{'struct'}."</h2>\n"; > + print "<h2>".$args{'type'}." ".$args{'struct'}. " - " .$args{'purpose'}."</h2>\n"; > print "<b>".$args{'type'}." ".$args{'struct'}."</b> {<br>\n"; > foreach $parameter (@{$args{'parameterlist'}}) { > if ($parameter =~ /^#/) { > @@ -498,8 +498,8 @@ sub output_function_html(%) { > my %args = %{$_[0]}; > my ($parameter, $section); > my $count; > - print "<h2>Function</h2>\n"; > > + print "<h2>" .$args{'function'}." - ".$args{'purpose'}."</h2>\n"; > print "<i>".$args{'functiontype'}."</i>\n"; > print "<b>".$args{'function'}."</b>\n"; > print "("; >
Attachment:
signature.asc
Description: This is a digitally signed message part
- References:
- [PATCH] kernel-doc: include struct short description in title output
- From: Randy Dunlap <[email protected]>
- Re: [PATCH] kernel-doc: include struct short description in title output
- From: Johannes Berg <[email protected]>
- [PATCH v2] kernel-doc: include struct short description in title output
- From: Randy Dunlap <[email protected]>
- [PATCH] kernel-doc: include struct short description in title output
- Prev by Date: Re: /proc/sys/net/ipv[46]/conf/ issue unsolved
- Next by Date: Re: somebody dropped a (warning) bomb
- Previous by thread: [PATCH v2] kernel-doc: include struct short description in title output
- Next by thread: [PATCH] fb: SM501 framebuffer driver
- Index(es):
 |