markup.ml

markup.ml copied to clipboard

Hi! I am using tyxml and ran into the same issue as https://github.com/ocsigen/tyxml/issues/307, but with the attribute `viewBox` instead. It only occurs with the ppx so I looked into the...

maurobringolf

Some invalid HTML documents yield Failure("require_current_element: None")

3

The following document (reduced as much as possible) makes ```parse_html``` raise a failure ```"require_current_element: None"```. Note that the document is indeed invalid since ```xmp``` elements are both deprecated and should...

Tchou

See https://discuss.ocaml.org/t/how-to-decode-html-code-while-encode-in-shift-jis/7634

aantron

Don't pretty-print SVG or MathML

1

See https://github.com/dmbaturin/soupault/issues/25: > This is arguably an upstream bug. Lambda Soup's (that is, Markup's) pretty-printer is supposedly sensitive to the meaning of elements, and should only insert whitespace where it...

aantron

aantron

Reverse operation for strings_to_byte

8

Hello there, I am trying to use Markup with Cohttp_lwt to do some HTML rewriting. I followed the example to get it to parse, but I am running into some...

xguerin

Stack overflow in js_of_ocaml

9

Reported by @Armael in https://github.com/aantron/markup.ml/issues/22#issuecomment-313258936: > Hi. I'm not sure what the state of this discussion is, but I can report on my recent experience of using markup with js_of_ocaml...

aantron

Introduce continuations, e.g. by linking to a good explanation

2

Markup.ml is written in continuation-passing style. For the vast majority of programmers on Earth, this means "using callbacks." So, I suggest renaming [`'a cont`](https://github.com/aantron/markup.ml/blob/1a33cc05af9e53a8efb21f43e963efa59e620ace/src/markup_common.ml#L17) to `'a callback` to make the...

aantron

Compile to JS and publish in NPM

18

It would be kind of cool if Markup.ml turned out to be the fastest HTML parser in JS – good PR for OCaml/ReasonML. But I don't know if this can...

aantron

Consider switching to string streams instead of char streams

3

The performance will probably be much better, but compare and measure first.

aantron