Skip to content

parser.mly: minor mistake in position of a symbol#32

Closed
let-def wants to merge 1 commit intoocaml:trunkfrom
let-def:fix-this-offset
Closed

parser.mly: minor mistake in position of a symbol#32
let-def wants to merge 1 commit intoocaml:trunkfrom
let-def:fix-this-offset

Conversation

@let-def
Copy link
Contributor

@let-def let-def commented Apr 11, 2014

This is not even a bug, just a behavior that differs from the rest of the code: usually in error productions, reported messages refers to the position of the error token. This is not the case here.

@gasche
Copy link
Member

gasche commented Apr 11, 2014

Thanks, merged in trunk.

@gasche gasche closed this Apr 11, 2014
@let-def let-def deleted the fix-this-offset branch November 15, 2014 13:30
lpw25 pushed a commit to lpw25/ocaml that referenced this pull request Jun 12, 2016
Gbury pushed a commit to Gbury/ocaml that referenced this pull request Aug 21, 2019
lthls pushed a commit to lthls/ocaml that referenced this pull request Dec 19, 2019
Gbury added a commit to Gbury/ocaml that referenced this pull request Mar 11, 2020
lthls pushed a commit to lthls/ocaml that referenced this pull request Apr 15, 2020
anmolsahoo25 pushed a commit to anmolsahoo25/ocaml that referenced this pull request Aug 25, 2020
lthls pushed a commit to lthls/ocaml that referenced this pull request Sep 23, 2020
lthls pushed a commit to lthls/ocaml that referenced this pull request Sep 23, 2020
lthls pushed a commit to lthls/ocaml that referenced this pull request Sep 24, 2020
poechsel pushed a commit to poechsel/ocaml that referenced this pull request Jul 2, 2021
stedolan pushed a commit to stedolan/ocaml that referenced this pull request Sep 21, 2022
ac520fe Merge flambda-backend changes
45cbc7d Turn "include functor" extension on by default (ocaml#38)
1531a02 Add the "include_functor" extension. (ocaml#32)
67b25c1 Merge flambda-backend changes

git-subtree-dir: ocaml
git-subtree-split: ac520fe
sadiqj pushed a commit to sadiqj/ocaml that referenced this pull request Feb 21, 2023
This extension adds the structure item form "include functor F", where F must be
a functor whose parameter can be "filled in" with the previous contents of the
current module.  This eliminates a common source of boilerplate where modules
create submodules that duplicate some of their contents and exist only for the
purpose of applying a functor.  For example:

    module Foo = struct
      module T = struct
        type t = int [@@deriving compare]
      end

      include T
      include Comparable.Make(T)
    end

Can now be written:

    module Foo = struct
      type t = int [@@deriving compare]

      include functor Comparable.Make
    end

Where the only difference is that the latter Foo doesn't have the submodule T.
EmileTrotignon pushed a commit to EmileTrotignon/ocaml that referenced this pull request Jan 12, 2024
* convert "support" page to use markdown; add next-mdx-remote bindings, simplify markdown and mdx modules
* Also: remove vercel config to reduce number of configs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants