caml-list - the Caml user's mailing list
 help / color / mirror / Atom feed
From: Martin Riener <martin@derivation.org>
To: caml-list@inria.fr
Subject: Re: [Caml-list] Syntax puzzle / suggestion
Date: Fri, 9 Aug 2019 16:48:34 +0200	[thread overview]
Message-ID: <dbb8c10d-0b83-071e-828e-2aaeec6cef86@derivation.org> (raw)
In-Reply-To: <20190809142719.ff4kogz4fq6r7vfr@annexia.org>


[-- Attachment #1.1: Type: text/plain, Size: 1700 bytes --]

I always found

  let [item1; item2] as all_items = ["hello"; "world" ]

readable but the compiler does does not infer that this match always works and
emits:

Warning 8: this pattern-matching is not exhaustive.
Here is an example of a case that is not matched:
(_::_::_::_|_::[]|[])

I've never touched the compiler so I don't have an idea how much effort it would
be to make the inference.

cheers,
Martin



On 8/9/19 4:27 PM, Richard W.M. Jones wrote:
> Let's imagine you have a list of named things, but you also want to
> collect them up into a single list.  For example:
> 
>   let item1 = "hello"
>   let item2 = "world"
>   let all_items = [ item1; item2 ]
> 
>   let () =
>     printf "item1 = %s\n" item1;
>     printf "all_items = %s\n" (String.concat ", " all_items)
> 
> This is fine, but there's a danger that a programmer could add item3
> but forget to add it to the "all_items" list.  (In the real world
> problem to which this applies, my items are complex and lengthy
> structures, and the "all-things-list" is well off the bottom of the
> page when you're viewing the top item).
> 
> My idea to fix this was to write:
> 
>   let all_items = [
>     ("hello" as item1);
>     ("world" as item2);
>   ]
> 
> Actually I was slightly surprised to find this doesn't compile.  I
> guess because "as" can only be used like this in patterns, not
> expressions.  Also the scoping is wrong because the "as item1" if it
> worked probably wouldn't apply outside the list.
> 
> I haven't worked out if it's possible to write this in ordinary OCaml,
> although I suppose ppx could solve it.  Any ideas if this is possible?
> 
> Rich.
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  parent reply	other threads:[~2019-08-09 14:48 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-09 14:27 Richard W.M. Jones
2019-08-09 14:40 ` Gabriel Scherer
2019-08-09 14:45   ` Richard W.M. Jones
2019-08-09 14:48 ` Martin Riener [this message]
2019-08-10  5:19   ` rixed
2019-08-09 14:51 ` Josh Berdine
2019-08-10 15:18   ` Oliver Bandel
2019-08-10 20:34   ` Chet Murthy
2019-08-09 14:56 ` Frédéric Bour

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=dbb8c10d-0b83-071e-828e-2aaeec6cef86@derivation.org \
    --to=martin@derivation.org \
    --cc=caml-list@inria.fr \
    /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).