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 --]
next prev 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