caml-list - the Caml user's mailing list
 help / color / mirror / Atom feed
From: Jacques Garrigue <garrigue@math.nagoya-u.ac.jp>
To: Jacques Carette <carette@mcmaster.ca>
Cc: OCaML List Mailing <caml-list@inria.fr>
Subject: Re: [Caml-list] Option to fully expand types in error messages?
Date: Sat, 4 Jun 2016 15:27:23 +0900	[thread overview]
Message-ID: <1F323615-4222-44E1-8188-EC829A45652A@math.nagoya-u.ac.jp> (raw)
In-Reply-To: <F7B18907EEED5244A3608C61DF32A4E410D5B39A@FHSDB2D11-2.csu.mcmaster.ca>

I see.
This is a problem with mismatching signatures.
This is much less well supported than unification errors (internally, the function called is moregeneral, which does not rely on unification).
This has already been much improved by tracking the mismatching item better, and -short-paths helps also by normalizing some types, but we should also track mismatches inside type schemes.

As you suggest, another solution would be full expansion.
This is a bad idea with objects, but may make sense for programs not using recursive types.
Note however that there is no function doing this kind of expansion inside the compiler...

Jacques

On 2016/06/04 00:42, "Carette, Jacques" wrote:
> 
> So here is an actual example.  The error I get is
> ocamlc -short-paths -c reproduce.ml 
> File "reproduce.ml", line 148, characters 21-25:
> Error: Signature mismatch:
>       ...
>       Values do not match:
>         val traverseexercise :
>           'a container PseudoCode.abstract ->
>           ('a PseudoCode.abstract ->
>            'b PseudoCode.abstract -> ('c * 'b) PseudoCode.abstract) ->
>           'b PseudoCode.abstract ->
>           'd container ->
>           ('d container -> 'c container PseudoCode.abstract -> 'e) -> 'e
>       is not included in
>         val traverseexercise :
>           loopdata container PseudoCode.abstract ->
>           (loopdata PseudoCode.abstract ->
>            loopdata PseudoCode.abstract ->
>            (loopdata * loopdata) PseudoCode.abstract) ->
>           loopdata PseudoCode.abstract ->
>           (< answer : 'a; state : 'b; .. >, loopdata) PseudoCode.cmonad
>       File "reproduce.ml", line 111, characters 6-206: Expected declaration
>       File "reproduce.ml", line 125, characters 8-24: Actual declaration
> 
> I can't tell from the above error what the actual cause is.  The full code is attached.  [This code is quite reduced already.  It is kept at this size to show in more detail the kinds of errors we're seeing.]
> 
> Any 'hint' from OCaml as to the precise nature of the non-match would sure be appreciated.
> 
> Jacques
> 
> ________________________________________
> From: Jacques Garrigue [garrigue@math.nagoya-u.ac.jp]
> Sent: June 2, 2016 19:59
> To: Carette, Jacques
> Cc: OCaML List Mailing
> Subject: Re: [Caml-list] Option to fully expand types in error messages?
> 
> On 2016/06/03 04:18, "Carette, Jacques" wrote:
>> 
>> In writing some code which uses a lot of monads with underlying types which use constraints, even simple errors can lead to extremely hard to read error messages.  The main reason is that the two types given in errors are partially expanded, to different levels.  This frequently means that the part where the type checker detects a mismatch is (extremely) opaque to human eyes.
>> 
>> In that case, it would actually be preferable to fully expand the types.  Yes, that will produce wallpaper.  But at least the mismatch should be considerably easier to catch.
>> 
>> Does this already exist, or should I submit a feature request?
>> 
>> Jacques
> 
> In the error message, types are expanded just enough to get down to the conflict.
> If the conflict is not visible at that point, this is probably a scoping error (and there should be an extra line stating that);
> otherwise this should be seen as a bug.
> As Yaron pointed, -short-paths can help by at least giving a normal form for paths (which may not be the expansion, but should be unique in the error context). But it will not expand a type if the expansion is not a type constructor, or if the parameters are different.
> 
> Jacques




      parent reply	other threads:[~2016-06-04  6:27 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-02 19:18 Carette, Jacques
2016-06-02 19:44 ` Gabriel Scherer
2016-06-02 19:50   ` Yaron Minsky
2016-06-02 21:59     ` Carette, Jacques
2016-06-02 23:59 ` Jacques Garrigue
2016-06-03 15:42   ` Carette, Jacques
2016-06-03 15:50     ` Yaron Minsky
2016-06-04  6:27     ` Jacques Garrigue [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1F323615-4222-44E1-8188-EC829A45652A@math.nagoya-u.ac.jp \
    --to=garrigue@math.nagoya-u.ac.jp \
    --cc=caml-list@inria.fr \
    --cc=carette@mcmaster.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).