Re: FW: Some comments on HTML tidy

Hi Reece,

Hey, thanks for catching these errors.  I'll try to update SF 
(http://sourceforge.net/projects/tidywrap) in the next couple days.

take it easy,
Charlie

At 08:30 PM 12/31/2005 +0000, you wrote:
>Hi Charles,
>
>I have posted some comments about your C++ HTML tidy binding to the 
>Tidy-dev mailing list. Here they are in case you missed them:
>
>>From: "Reece Dunn" <[email protected]>
>>To: [email protected]
>>Subject: [Tidy-dev] Some comments on HTML tidy
>>Date: Sat, 31 Dec 2005 19:33:03 +0000
>>
>>Hi all,
>>
>>I am using HTML tidy as a HTML parser that feeds into a text outputter to 
>>do HTML ==> text conversion. This is a great tool, but here are a few 
>>things I have noticed and have modified:
>>
>>[1]  in the C++ binding (tidyx.h), IsHeader() is missing a return statement:
>>-    Bool IsHeader()      { tidyNodeIsHeader( tnod() ); } /* h1, h2, ... */
>>+    Bool IsHeader()      { return tidyNodeIsHeader( tnod() ); } /* h1, 
>>h2, ... */
>>
>>[2]  the C++ binding functions should be const qualified, e.g.
>>
>>    Bool IsHTML() const       { return tidyNodeIsHTML( tnod() ); }
>>
>>If people are interested in these modifications, I am willing to post the 
>>patch files for them.
>>
>>Happy New Year,
>>- Reece H. Dunn
>
>- Reece
>



-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click