doxygen-develop

[Doxygen-develop] "\c \e foo bar" = bug?

[Matthew W. <mw_...@us...>]

Is this a bug?

"\c \e foo bar" == "<tt><i>foo</i> bar</tt>"

Shouldn't it be:

"<tt><i>foo</i></tt> bar"?

...and any pointers where I can go tweak to fix it?

-- 
Matthew
"Do you do windows as well?"
"Only when I'm forced to deal with Microsoft..."
   -- from a story by Feech

Re: [Doxygen-develop] "\c \e foo bar" = bug?

[Dimitri v. H. <do...@gm...>]

On 3/7/07, Matthew Woehlke <mw_...@us...> wrote:
>
> Is this a bug?
>
> "\c \e foo bar" == "<tt><i>foo</i> bar</tt>"
>
> Shouldn't it be:
>
> "<tt><i>foo</i></tt> bar"?



That would indeed be more logical.


...and any pointers where I can go tweak to fix it?



src/docparser.cpp is where the commands are handled, in particular
look at defaultHandleToken() and handleStyleArgument()

Regards,
   Dimitri

Re: [Doxygen-develop] "\c \e foo bar" = bug?

[Matthew W. <mw_...@us...>]

Dimitri van Heesch wrote:
> On 3/7/07, Matthew Woehlke wrote:
>> Is this a bug?
>>
>> "\c \e foo bar" == "<tt><i>foo</i> bar</tt>"
>>
>> Shouldn't it be:
>>
>> "<tt><i>foo</i></tt> bar"?
> 
> That would indeed be more logical.
> 
>> ...and any pointers where I can go tweak to fix it?
> 
> src/docparser.cpp is where the commands are handled, in particular
> look at defaultHandleToken() and handleStyleArgument()

Thanks. Oh, and I saw your reply to Tim, I'll go file a Real Bug Report 
now... hopefully I can get a patch in a few days too. I'm still on 1.4.7 
because I want to use the @li2 I wrote. (Btw, what ever happened with 
that patch I sent?)

-- 
Matthew
Caution: keep out of reach of adults.

Re: [Doxygen-develop] "\c \e foo bar" = bug? [PATCH]

[Matthew W. <mw_...@us...>]

Matthew Woehlke wrote:
> Dimitri van Heesch wrote:
>> On 3/7/07, Matthew Woehlke wrote:
>>> Is this a bug?
>>>
>>> "\c \e foo bar" == "<tt><i>foo</i> bar</tt>"
>>>
>>> Shouldn't it be:
>>>
>>> "<tt><i>foo</i></tt> bar"?
>> That would indeed be more logical.
>>
>>> ...and any pointers where I can go tweak to fix it?
>> src/docparser.cpp is where the commands are handled, in particular
>> look at defaultHandleToken() and handleStyleArgument()
> 
> Thanks. Oh, and I saw your reply to Tim, I'll go file a Real Bug Report 
> now... hopefully I can get a patch in a few days too. I'm still on 1.4.7 
> because I want to use the @li2 I wrote. (Btw, what ever happened with 
> that patch I sent?)

Here's a patch; I think it qualifies as "an ugly hack" :-). I added this 
to http://bugzilla.gnome.org/show_bug.cgi?id=418615 as well, but it 
would be great if other doxygen hackers could take a look. Be sure to 
add RetVal_Nested to doctokenizer.h also (yeah, I'm lazy, diff -r would 
have pulled in all my other changes also so I left out the one-liner).

Thanks again for the pointer, Dimitri, you were spot-on! I also touched 
handleCommand as well although I'm not sure that is necessary.

-- 
Matthew
Caution: keep out of reach of adults.