[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: KDE Documentation & deprecated API
From:       Andreas Simon <yuipx () gmx ! net>
Date:       2002-02-26 15:42:40
[Download RAW message or body]

On Tue, 2002-02-26 at 15:53, Chris Kuhi wrote:
> On Tuesday 26 February 2002 14:20, Andreas Simon wrote:
> > On Tue, 2002-02-26 at 13:37, gordon.machel@t-online.de wrote:
> > > I'm currently browsing through the API docs as well,
> > > as I'm preparing them for the use with doxygen. The
> > > reason for the observed behaviour is that sometimes
> > > the @deprecated command is followed by a '.' or a ',',
> > > thereby preventing the parser from recognizing the
> > > command.
> >
> > Yes, line 88 in kdoc/src/kdocParseDoc.pm is obviously wrong:
> >
> > 	elsif ( $text =~ /^\s*\@deprecated\s*/ ) {
> >
> > I guess it should better be
> >
> > 	elsif ( $text =~ /^\s*\@deprecated[:punct:]?\s*/ ) {
> >
> > (my perl is a little bit rusty) Does anyone want to 
> commit?
> > Or should the ...
> 
> That line will *not* prevent a match on  '@deprecated.' or in fact 
> anything following the '@deprecated' since it's only trying to match 
> zero or more spaces after, and there *are* zero spaces.  See for 
> yourself:

Of course you're right. Actually, I haven't even really looked
at the problem. Maybe I should first build kdoc ;-)

Cheers,
Andreas Simon


 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]