[erlang-patches] Return end locations in erl_scan
Anthony Ramine
n.oxyde@REDACTED
Wed Mar 20 13:44:28 CET 2013
Hi,
Replying on list because I think it's important.
As I said to someone I don't remember the name, this patch is only a
necessary step to what my final goal is: Clang-like diagnostics for
Erlang compilation [1]. Is that something the OTP team wouldn't like
to see?
How is the end location in tokens redundant? I need the end locations
of each tokens to be able to compute the location ranges of each node
in the AST, see my work-in-progress commit for more informations [2].
That being said, I am interested in having your feedback about the
implementation.
Regards,
[1] http://clang.llvm.org/diagnostics.html
[2] https://github.com/nox/otp/commit/2c8038c#diff-1
PS: Sorry Hans for replying twice, I failed the Cc header.
--
Anthony Ramine
Le 20 mars 2013 à 13:23, Hans Bolinder a écrit :
> Hi Anthony,
>
> Sorry for not replying sooner.
>
> We'll most likely reject you patch. I asked Vlad Dumitrescu about it,
> and he agrees with me that the functionality (the end location of
> tokens) is redundant.
>
> Apart from that: when it comes to the implementation there are a few
> things I don't approve of, but I need to take a closer look before
> saying anything more. You've put in a good effort here, and I intend
> to elaborate a little more on the implementation when I find the time
> to investigate in more detail.
>
> Best regards,
>
> Hans Bolinder, Erlang/OTP team, Ericsson
More information about the erlang-patches
mailing list