Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

allow M.begin end to mean local open #6740

Closed
vicuna opened this issue Dec 30, 2014 · 1 comment
Closed

allow M.begin end to mean local open #6740

vicuna opened this issue Dec 30, 2014 · 1 comment
Labels
feature-wish wontfix

Comments

@vicuna
Copy link

vicuna commented Dec 30, 2014

Original bug ID: 6740
Reporter: maverickwoo
Status: closed (set by @mshinwell on 2016-12-08T15:27:45Z)
Resolution: won't fix
Priority: normal
Severity: feature
Version: 4.02.1
Category: ~DO NOT USE (was: OCaml general)
Monitored by: @hcarty

Bug description

At present we can use M.( ) to mean local open. Since begin/end and (/) are compatible in most other places, it seems that allowing M.begin end is natural too.

Thank you for your consideration.
@vicuna
Copy link
Author

vicuna commented Dec 8, 2016

Comment author: @mshinwell

I asked three other developers about this and opinions were mixed:

  1. "Just weird"
  2. In favour
  3. Unsure. (Seems a bit odd.)

Given there have been no comments for two years I'm going to close this---but please feel free to open a Github pull request containing your proposed implementation for this feature.

@vicuna vicuna closed this as completed Dec 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-wish wontfix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vicuna