TODO 14.2 KB
Newer Older
POTTIER Francois's avatar
POTTIER Francois committed
1 2 3 4 5
* About --list-errors and --interpret-error and --compile-errors:
  document these options
    explain that any production that contains [error] is ignored by --list-errors
  document the .messages file format
  document the workflow
6
  collect performance data
POTTIER Francois's avatar
POTTIER Francois committed
7 8 9 10 11 12 13 14 15 16 17 18 19
    correlate with star size and alphabet size, etc.
    create separate graphs for 3 modes: --lalr, pager, --canonical
  Modify LRijkstra to produce a valid .messages file
  Implement --compare-errors to make sure that every error is covered.
  In Menhir's bootstrapped parser:
    re-establish rule_specific_token?
      git show 64504bd5214794dc34530f93faccd9f62f049dcf
    use a canonical automaton?

* When dealing with errors, should we back up to the last shift action,
  undoing any non-canonical reduce actions? if so, a lot of code is
  impacted (--list-errors, --interpret-error, --compile-errors, --compare-errors,
  plus fancyDriver.ml).
POTTIER Francois's avatar
POTTIER Francois committed
20

POTTIER Francois's avatar
POTTIER Francois committed
21 22 23 24
* Dans les avantages de Menhir versus ocamlyacc (dans la doc et
  sur la page Web), ajouter le back-end Coq, l'API incrémentale
  et l'API d'inspection, les règles anonymes...

POTTIER Francois's avatar
POTTIER Francois committed
25 26
* bench/bad/rule-error.mly should produce a syntax error earlier?

27 28 29 30
* BUG dans ErrorReporting si l'erreur se produit dans l'état
  initial. Pénible que l'API d'inspection ne donne pas accès
  à l'état courant lorsque la pile est vide (état initial donc).
  Pourrait-on exposer cet état? Polymorphe? incoming_symbol
POTTIER Francois's avatar
POTTIER Francois committed
31
  devrait alors produit une option. Et le cas Nil du type
32
  stream devrait contenir l'état initial...
POTTIER Francois's avatar
POTTIER Francois committed
33
  attention, car si on lui applique la fonction items, on va
34 35 36 37
  obtenir un item spécial S' -> S # qui va poser problème.
  Exposer l'état initial avec un type différent lr1initial?
  De toute façon il faudra compliquer la notion de explanation
  qui ne pourra plus toujours être basée sur un item...
POTTIER Francois's avatar
POTTIER Francois committed
38

39
* Gabriel veut pouvoir inclure MenhirLib dans le parser engendré
POTTIER Francois's avatar
POTTIER Francois committed
40

41 42 43
* gros BUG à corriger: il ne faut pas exiger le prochain token
  AVANT une réduction par défaut. Il faut suivre ocamlyacc et
  bison. Réfléchir... et corriger les deux back-ends. Attention
44 45
  toutefois, c'est un changement incompatible. Option de ligne
  de commande?
46
  Réfléchir à l'impact sur les positions.
POTTIER Francois's avatar
POTTIER Francois committed
47
  Il faut aussi revisiter la notion de conflit end-of-stream.
48 49 50 51
  - si réduction par défaut, alors pas de conflit possible!
  - si conflit, alors pourquoi on le résoud en éliminant la
    réduction sur #? on pourrait faire aussi le choix opposé,
    à savoir éliminer toutes les autres actions. Que fait
POTTIER Francois's avatar
POTTIER Francois committed
52
    ocamlyacc?
53

POTTIER Francois's avatar
POTTIER Francois committed
54 55
* autoriser %token FOO "foo"
  pour pouvoir afficher les tokens sous forme plus lisible
56
  et auto-générer une fonction (ou une table) print_terminal
POTTIER Francois's avatar
POTTIER Francois committed
57

POTTIER Francois's avatar
POTTIER Francois committed
58
* autoriser %token FOO (unused)
59 60
  (syntaxe concrète à déterminer)
  équivalent à --unused-token FOO sur la ligne de commande.
POTTIER Francois's avatar
POTTIER Francois committed
61

62
* Produce well-chosen (predictable) names for anonymous rules?
POTTIER Francois's avatar
POTTIER Francois committed
63

64 65
* In the standard library, possibly rename "anonymous" and "embedded" and
  document them. The non-inline version allows embedding an action in the
66
  middle of a rule.
67

POTTIER Francois's avatar
POTTIER Francois committed
68 69 70 71
* Should we always avoid querying the lexer before a default reduction?
  Would that be difficult/possible? Would that help people who want to
  trigger changes in the lexer?

POTTIER Francois's avatar
POTTIER Francois committed
72 73
* ErrorReporting: experiment with merging several symbols
  so as to reduce the noise (e.g. operators, expressions,
POTTIER Francois's avatar
POTTIER Francois committed
74 75
  etc.). Or just print the same way, but don't quotient
  internally, as the locations are not the same.
POTTIER Francois's avatar
POTTIER Francois committed
76

POTTIER Francois's avatar
POTTIER Francois committed
77 78
  Also, sort the explanations by a certain measure of
  complexity. E.g., those that go back to a "higher" non-terminal
POTTIER Francois's avatar
POTTIER Francois committed
79 80 81 82 83 84 85
  symbol should be preferred. Or those that go back further away
  in the buffer?

  Think about HTML output with mouse-over. Or mouse-click.
  Find a way of solving the scrolling problems.
    <pre> and <table> side by side, each with its scroll bar,
    would be nice.
POTTIER Francois's avatar
POTTIER Francois committed
86

POTTIER Francois's avatar
POTTIER Francois committed
87 88
* Add left-recursive lists to the standard library.

89 90 91 92
* Define [print_result].
  Define a printer that shows the states in the stack.
  Try computing a set of expected symbols in the current state.

POTTIER Francois's avatar
POTTIER Francois committed
93
* Generate default printers for terminal and nonterminal.
POTTIER Francois's avatar
POTTIER Francois committed
94 95
  Using modular implicits, generate terminal2token and
  nonterminal2value.
POTTIER Francois's avatar
POTTIER Francois committed
96

POTTIER Francois's avatar
POTTIER Francois committed
97 98 99 100
* document that --depend may produce inaccurate dependencies
  for parser.cmi and that it is recommended to use --raw-depend
  --ocamldep "ocamldep -modules" and work from there (which is
  what ocamlbuild does).
101

POTTIER Francois's avatar
POTTIER Francois committed
102
* Confirm and understand why the code back-end slows down by 50%
POTTIER Francois's avatar
POTTIER Francois committed
103 104
  between 4.01.0 and 4.02.1. Use the tag last309 for this purpose,
  or the master branch.
POTTIER Francois's avatar
POTTIER Francois committed
105

106 107 108
* Suite des patchs de Frédéric Bour.
    API d'inspection complète.
      Exposer le nombre d'états (pour la mémoisation) et une conversion état -> entier.
POTTIER Francois's avatar
POTTIER Francois committed
109
      Idem pour les productions.
POTTIER Francois's avatar
POTTIER Francois committed
110 111
      Fonctions d'affichage pour les types terminal, nonterminal, etc.?
      Fonctions d'isomorphisme entre token et 'a terminal * 'a?
POTTIER Francois's avatar
POTTIER Francois committed
112
    Librairie(s) pour la gestion des erreurs.
113
    Librairie pour la complétion du parsing.
POTTIER Francois's avatar
POTTIER Francois committed
114
      Need a way of feeding the parser a nonterminal symbol.
POTTIER Francois's avatar
POTTIER Francois committed
115
      Or a way of applying a semantic action to a vector of semantic values.
116 117

* Meilleurs messages d'erreur de syntaxe.
118 119 120 121 122
    Revenir en arrière après une erreur, car l'erreur
    n'est détectée qu'après quelques réductions. Une fois que l'on est
    revenu, on peut examiner les items de l'état courant et donner des
    positions de début d'item qui devraient être intéressantes.    
  Possibilité de coloriser ce qu'on a lu en indiquant ce qu'on a compris
POTTIER Francois's avatar
POTTIER Francois committed
123
  (sortie HTML).
POTTIER Francois's avatar
POTTIER Francois committed
124

POTTIER Francois's avatar
POTTIER Francois committed
125 126 127
* Test and document --lalr (maybe). Explain that it is (probably)
  incompatible in principle with --explain. Modify the code to fail
  gracefully when the problem arises.
POTTIER Francois's avatar
POTTIER Francois committed
128

129 130 131
* les positions fournies par menhir ne sont pas les mêmes fournies par
  ocamlyacc (voir messages de Tiphaine Turpin à partir du 30/08/2011).
  Est-ce un problème? Peut-on documenter quelles sont les
132
  positions fournies par Menhir? En particulier, pour une production
133 134 135 136 137 138 139 140
  vide, Menhir utilise _menhir_env.lexer.lex_start_p, c'est-à-dire le
  début du dernier token lu par le lexer; en principe c'est le token
  de lookahead donc ça va, mais ça pourrait donner un résultat bizarre
  si on réduit par défaut une production vide (on a alors pour position
  le début du token précédent?) (BUG?). Par ailleurs, il faut être
  conscient que si l'action ocamllex se rappelle récursivement, alors
  _menhir_env.lexer.lex_start_p n'est pas le début du token mais une
  position quelque part à l'intérieur du token (e.g. après des espaces
141 142 143 144
  blancs). SUGGESTION DE SOLUTION: au lieu de paires (startpos, endpos),
  utiliser un type abstrait d'intervalles, avec un traitement particulier
  de l'intervalle vide. (Voir mon message du 15/09/2011.)

145 146 147 148 149
* Tirer au clair la sémantique des $startpos/$endpos sur les productions vides.
    Jacques-Henri dit que $startpos devrait toujours être $endpos du symbole précédent plus les blancs,
    et symétriquement pour $endpos. Mais pour implémenter ça il faut consulter la pile?
  BUG: %inline ne préserve pas la sémantique de $startpos/$endpos.
    C'est vrai pour les productions epsilon (forcément)
150
    mais aussi pour des productions non-epsilon
151 152
      car $startpos de la production inlinée est changée en $endpos(x) où x est le symbole précédent!
  Si on gérait les positions par macro-expansion *avant* de faire l'inlining
153 154
  alors la passe d'inlining n'aurait pas besoin de s'en occuper (et serait
  correcte par construction).
POTTIER Francois's avatar
POTTIER Francois committed
155 156 157

* Tenter une minimisation a posteriori de l'automate. Sur la grammaire
  OCaml on devrait retrouver l'automate LALR, non?
158 159 160
  Faire cette minimisation *après* la résolution des conflits afin que
  la minimisation n'affecte pas le comportement de l'automate, même en
  présence de directives de priorité. Du coup si on combine --canonical
POTTIER Francois's avatar
POTTIER Francois committed
161 162 163 164 165
  et --minimize alors on devrait obtenir un automate correct, proche de
  IELR, voir Denny et Malloy. Confirmer.

* Pourquoi --canonical --table ne marche pas sur une grosse grammaire?
  (3m57 pour ocaml.mly versus 16s sans --table)
166
  Afficher combien de temps est passé à comprimer les tables.
POTTIER Francois's avatar
POTTIER Francois committed
167 168

* TableBackend en principe ne devrait pas avoir besoin de Invariant?
169
  Gain de temps/simplicité possible.
POTTIER Francois's avatar
POTTIER Francois committed
170

171 172
* Could we avoid listing the contents of MenhirLib in 4 different files?

POTTIER Francois's avatar
POTTIER Francois committed
173 174 175
* bundle the Coq library with menhir, and add demos/calc-coq
  to show how it is used

POTTIER Francois's avatar
POTTIER Francois committed
176 177
* Clarifier si ocamlbuild doit recevoir -use-ocamlfind, -no-ocamlfind,
  ou rien; tester en particulier sous Windows?
POTTIER Francois's avatar
POTTIER Francois committed
178
  Modifier la doc pour recommander ocamlfind.
POTTIER Francois's avatar
POTTIER Francois committed
179

180 181
* ajouter une possibilité de désactiver certains warnings (utile pour
  le cours de compilation où on démarre avec une grammaire à trous),
fpottier's avatar
fpottier committed
182 183
  par exemple les non-terminaux inaccessibles

184
* move to a new license (GPL V2?), as per Hongbo Zhang's request.
185
* Les fichiers dans demos/ ne devraient pas être sous QPL mais libres...
186

fpottier's avatar
fpottier committed
187 188
* _stage2/menhir.native --interpret fancy-parser.mly
  crashes with Stack_overflow when the input is just "grammar:\n"
POTTIER Francois's avatar
POTTIER Francois committed
189
  ... it seems to loop now ...
190

191
* BUG: la directive $type<...> nt donne lieu à un message d'erreur bizarre
192 193
  si le non-terminal nt n'existe pas. (Jacques-Henri.)

194 195 196
* On pourrait rendre moins restrictif le critère qui garantit la terminaison
  de l'expansion des non-terminaux paramétrés. Il suffirait de vérifier que
  les paramètres effectifs sont toujours des atomes (symboles terminaux ou
197 198
  non-terminaux), non? (Jacques-Henri.)

POTTIER Francois's avatar
POTTIER Francois committed
199 200 201
* En mode code on tire les tokens d'un lexbuf et l'utilisateur doit construire
  l'API moderne par `wrapping', s'il la souhaite. Mais en mode table l'API
  moderne est native et l'API traditionnelle est construite (par nous) par
202 203 204 205
  `wrapping', d'où risque de `double wrapping' pour un utilisateur qui veut
  alterner entre code et table. On devrait exposer nous-mêmes les deux API.
  En profiter pour laisser l'utilisateur contrôler comment un token doit
  être emballé en `fat token' (token + range).
POTTIER Francois's avatar
POTTIER Francois committed
206

207 208 209
* %inline pourrait-il fonctionner quand les productions à inliner ont un %prec?
    (ça aurait un sens au moins quand on inline dans une production unité?)
    (ou plus généralement quand on inline en dernière position?)
POTTIER Francois's avatar
POTTIER Francois committed
210

211
* Reconnaître les directives # n "foo" dans le fichier .mly et en tenir
POTTIER Francois's avatar
POTTIER Francois committed
212 213
  compte dans les locations.

214 215 216 217 218
* BUG: message de Valentin Gatien-Baron du 09/01/2010: le bug de --explain
  est-il bien le bug connu? peut-on le corriger? ne suffirait-il pas de
  passer sous silence les conflits qui ont lieu dans une partie inaccessible
  de l'automate?

219 220
* BUG: message de Valentin Gatien-Baron du 09/01/2010: le warning erroné à
  propos d'un %prec inutile peut-il être évité?
221

222 223 224
* Implémenter un test de détection de boucles. Si la grammaire contient une
  boucle, elle est infiniment ambiguë. Pour le test, calculer tous les
  nullables, ajouter une production unité A -> B dès qu'il existe une
225
  production A -> alpha B beta avec alpha et beta nullables, et regarder
226
  ensuite s'il existe un cycle de productions unité.
227

POTTIER Francois's avatar
POTTIER Francois committed
228
* Permettre une trace avec autre chose que fprintf stderr. Cela permettrait
229 230 231
  au programmeur (par exemple) de décider à runtime s'il veut afficher ou
  non la trace. (En mode --trace.) Et si possible, donner accès depuis
  l'extérieur à la fonction print_token qui est engendrée.
232

233
* (petit) BUG: --follow-construction n'affiche pas le contenu des états
234 235 236
  initiaux (r0)

* relire la preuve de Pager et comprendre si ce que fait Lr1.grow est correct
237
  ou pas; faut-il faire grossir les états existants, ou bien (par exemple) les
238 239 240 241
  supprimer et recommencer leur construction?

* que fait/que doit faire la combinaison --tokens-only --depend?

242
* Préserver dirname/ dans les directives #line émises.
243

244 245
* BUG: Boris (1.- bouclage parser sur une entrée erronée 2.- une lecture
  trop loin dans le stream, également liée à error).
246

247 248
* éviter "Warning: 22 conflicts ..." car emacs prend "Warning" pour un
  nom de fichier; d'après Damien, il devrait suffire que la phrase après
249 250
  les deux points ne commence pas par un nombre.

251
* ajouter une option -q pour désactiver tous les warnings
252

253 254
* si une variable est inutilisée dans une action sémantique, le
  warning est affiché dans le code produit.
255

256 257 258 259
* si une parenthèse est mal fermée dans une action sémantique,
  l'erreur de syntaxe peut être détectée par OCaml en dehors
  des actions sémantiques. Bétonner en mettant une séquence
  improbable de begin/end/parenthèseses autour de l'action?
260 261 262

* BUG: solving a shift/reduce conflict in favor of reduction can
  cut a path that was required in order to explain another conflict.
POTTIER Francois's avatar
POTTIER Francois committed
263 264
  (see e.g. belloeil.mly) (et le reduced_parser.mly d'Adrien Guatto)
  Or just give up on explaining the conflict in this case.
265 266 267
  Take priorities into account *during* the construction of the
  automaton? Is it feasible? Compatible with Pager?

268 269
* BUG: cutdown.mly

270 271 272 273
* le fichier .conflicts devrait etre supprime par --explain s'il
  n'y a aucun conflit

* dans le dump de l'automate, rajouter les transitions qui ont
POTTIER Francois's avatar
POTTIER Francois committed
274
  ete supprimees par resolution de conflits. Afficher aussi les
275
  réductions par défaut?
276 277 278

* expliquer aussi les conflits end-of-stream

279
* Documenter l'emploi d'ocamlbuild.
280

281
* Lorsqu'on crée un pseudo-lexème pour utiliser dans les
282
  directives %prec, la question de savoir s'il est associatif
283
  à gauche ou à droite ou pas du tout ne se posera jamais. Donc,
284
  pour faire propre, on ne devrait pas autoriser l'emploi de
285 286
  %left, %right, %nonassoc (sauf pour compatibilité arrière,
  avec warning) mais introduire un nouveau mot-clé %fictitious
287 288
  ou %pseudo ou whatever.

289 290 291 292
* Autre remarque du même tonneau, si on souhaite choisir un
  niveau mais pas un statut d'associativité (parce qu'on pense
  qu'il est inutile), il faudrait pouvoir déclarer %neutral
  (whatever) et obtenir un niveau au statut non-spécifié (d'où
293 294
  erreur si on tente de consulter ce statut).

295 296 297
* Prévoir une sortie --ocamlyacc qui affiche la grammaire sans
  les liaisons des valeurs sémantiques et sans les actions
  sémantiques?
298

299 300 301 302 303
* Faire le nécessaire pour empêcher les warnings d'ocaml 3.09
  sur les valeurs sémantiques nommées et non utilisées? Le
  problème est que l'erreur est forcément rapportée dans le
  fichier .ml engendré. Et ça résoudrait le problème du cas
  où on n'utilise que $startpos(v) et pas v.
304 305

* Il faudrait activer le warning qui recommande d'utiliser --infer
306 307
  dès que la librairie standard est utilisée (et pas seulement lorsque
  %inline est utilisé).
308