Re: [Htmlparser-developer] Update (Claude - ur feedback needed)
Brought to you by:
derrickoswald
|
From: Somik R. <so...@ya...> - 2002-08-06 07:17:34
|
Hi Kaarle,
It seems like we may have acted hastily in correcting this (even in =
HTMLImageScanner). I just tried Claude's page again, and I find that the =
image is not parsed. Amit also mentioned sometime back that we ought to =
flag some kind of error.=20
Of course IE does not collapse- it continues parsing.=20
So - I think you should not put in this fix to parseParameters(). I =
should also rollback my fix and throw an error (?) - or probably throw a =
bad image tag, where you cannot retrieve the data.
OTOH - the other side of the coin is - if someday people decide to =
kick IE out, and write a new browser with this parser, such pages would =
work fine. In which case, it would be good to have fixes like this.
I find myself tilting to the former argument, however attractive the =
latter may sound. Amit, Claude--> what are your comments ?
Claude - as this bug was reported by you - I'd like to ask what do =
you expect ?
Regards,
Somik
----- Original Message -----=20
From: Kaarle Kaila=20
To: so...@ya... ; htm...@li...=20
Sent: Tuesday, August 06, 2002 4:07 PM
Subject: Re: [Htmlparser-developer] Update
I still had a look at the code and made a small addition
that would accept <a b"c"> as <a b=3D"c">
Would it be usefull to have it inserted into CVS?
or is it OK as it is?
regards
Kaarle
PS! I can't access CVS until the evening=20
---- Original Message ----
From: so...@ya...
To: htm...@li...
Subject: Re: [Htmlparser-developer] Update
Date: Tue, 6 Aug 2002 15:42:29 +0900
>Hi Kaarle,
> Thanks for the clarification.
>
>Regards,
>Somik
>
> >I did not really do that I think. I just made a testcase that=20
>seems=20
> >to verify=20
> >that <a b"c"> will be assume to be <a b> , same as <a b=3D"">
> >
> >Oh - then what happens to c, is it ignored?=20
> >
>
> Yes! That's what seems to happen. As I said I only added a testcase
> to verify what happens. I did not change the code for this purpose.
>
> regards
> Kaarle
>
>
>
> >Cheers,
> >Somik
> >
> -----------------------------
> Kaarle Kaila
> http://www.iki.fi/kaila
> mailto:kaa...@ik...
>
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> Htmlparser-developer mailing list
> Htm...@li...
> https://lists.sourceforge.net/lists/listinfo/htmlparser-developer
>
-----------------------------
Kaarle Kaila
http://www.iki.fi/kaila
mailto:kaa...@ik...
|