Re: Meaning and use of TIDY_STORE_ORIGINAL_TEXT

Hi Bjoern,

Thanks for your input ... I did a 'search' of the
tidy-cvs for TIDY_STORE_ORIGINAL_TEXT, and only one
entry was found, back in July last year, but it was
not directly related to this considerable body of
code ...

I am afraid I did not understand your -
>online CVS web installation to check who/when/why ...'
online - yes, CVS - yes, web - yes, installation - what
installation? ... what URI/URL? Exactly where else
are you suggesting I look?

On doing some further searching of archives,
I did note another had mentioned one of the 'bugs' my
fixes address back in May 2005 ... and noted your
reply to Hermann then, saying 'removed or re-designed'
after stating 'I haven't looked at the code' ...

I presently USE this 'feature' to add important
diagnostic output - it is great to be able to SEE the
actual input text Tidy is parsing ... then follow what
the code is doing with this input ... so I directly USE
this body of code now, but only as DIAGNOSTICS ...

Thus my minor fixes were small parts I noted were
'broken', perhaps, as you point out, by subsequent
changes in the code ... none of these 'fixes' would
in any way effect the current 'release' code, since
they are all under a compiler switch ... which is
OFF (not defined) by default ...

Also I am not sure why you characterise the code as -
>an ugly mess and had some very subtle bugs ...
Just not sure how code can be 'ugly'? Someone has
put good time and effort into building this code,
and it largely works as expected, and my 'fixes'
remove the not so subtle 'errors' that had either
crept into it, or perhaps were there from the
beginning ...

As mentioned, although I suspect it somehow misses
some line endings in certain circumstances, basically
it moves the 'store', built up during the token parsing
in to the 'node' tree, where it is fully released,
that is freed, on exit ... so there are no memory
allocation issues with it ...

I certainly agree -
>Tidy very conciously(sic) normalizes the spacing
and I, like you mention of others, would -
>like to retain their original spacing or would like
>more options to take more formatting choices into
>account, ...
and I think this very stable, and efficient 'original
store' may be ONE WAY to achieve SOME of that ...

But meantime, it is a excellent 'debug' aid, if
nothing else ;=)) Again, I hope my minor fixes can
make it into CVS ...

Regards,

Geoff.

EOF

_________________________________________________________________
ASUS M5 Ultra-slim lightweight is Now $1999 (was $2,999)  
http://a.ninemsn.com.au/b.aspx?URL=http%3A%2F%2Fwww%2Easus%2Ecom%2Eau%2F&_t=752129232&_r=Hotmail_tagline_23Nov05&_m=EXT



-------------------------------------------------------
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