Use XML::LibXML parser - Githubissues

kasei / perlrdf

Deprecated in favor of the Attean package

26 stars 25 forks source link

Use XML::LibXML parser #153

Closed kjetilk closed 6 years ago

kjetilk commented 7 years ago

Hi!

What do you think about doing this instead of using HTML::HTML5::Parser?

Kjetil

kasei commented 7 years ago

Sorry I didn't get to this sooner. I saw you mention something about this on irc. Did that issue get resolved? Do you think this can be merged without any issues?

kjetilk commented 7 years ago

We would need to do s/use HTML::HTML5::Parser/use XML::LibXML/ or something like that too, but yes, this should work.

The downside of doing this is that HTML::HTML5::Parser is more liberal as to what it accepts, but since this is unlikely to be used with random HTML from the Web, I think that is a small price to pay in this case.

kjetilk commented 7 years ago

Now, it should be in a releasable state. I think this is the best we can do.