You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Original bug ID: 801 Reporter: administrator Status: closed Resolution: not a bug Priority: normal Severity: minor Category: ~DO NOT USE (was: OCaml general)
Bug description
Salut,
le parser refuse ça:
type t = [ `A of int ]
type s = [ t ]
alors qu'il accepte ça:
type t = [ `A of int ]
type s = [ | t ]
(dans le premier cas, il s'attend à un tag_field, alors que c'est
un simple_core_type2; je n'ai pas testé, mais mettre row_field
à la place de tag_field dans LBRACKET tag_field RBRACKET,
ça devrait faire l'affaire).
--
Alain
The text was updated successfully, but these errors were encountered:
le parser refuse ça:
type t = [ `A of int ]
type s = [ t ]
alors qu'il accepte ça:
type t = [ `A of int ]
type s = [ | t ]
(dans le premier cas, il s'attend à un tag_field, alors que c'est
un simple_core_type2; je n'ai pas testé, mais mettre row_field
à la place de tag_field dans LBRACKET tag_field RBRACKET,
ça devrait faire l'affaire).
Ce n'est pas un bug. Accepter le premier cas causerait une ambiguite'
dans les types de classes (le [ y est utilise' pour indiquer les
parametres d'un type de classe par opposition a` un type normal).
On pourrait eventuellement refuser le deuxieme cas aussi, comme
inutile (tout ca est equivalent a` type s = t), mais ca ne m'a pas
paru justifier une complication supplementaire dans le parser.
Original bug ID: 801
Reporter: administrator
Status: closed
Resolution: not a bug
Priority: normal
Severity: minor
Category: ~DO NOT USE (was: OCaml general)
Bug description
Salut,
le parser refuse ça:
type t = [ `A of int ]
type s = [ t ]
alors qu'il accepte ça:
type t = [ `A of int ]
type s = [ | t ]
(dans le premier cas, il s'attend à un tag_field, alors que c'est
un simple_core_type2; je n'ai pas testé, mais mettre row_field
à la place de tag_field dans LBRACKET tag_field RBRACKET,
ça devrait faire l'affaire).
--
Alain
The text was updated successfully, but these errors were encountered: