ocamldep does not handle SEMISEMI correctly

[vicuna]

Original bug ID: 7793
Reporter: @hhugo
Assigned to: @nojb
Status: closed (set by @nojb on 2018-05-23T23:10:02Z)
Resolution: fixed
Priority: normal
Severity: minor
Category: tools (ocaml{lex,yacc,dep,debug,...})
Monitored by: @nojb

Bug description

The following program is accepted by the ocaml compiler but not by ocamldep

;;

;;2

[vicuna]

Comment author: @hhugo

ocaml-ppx/ocamlformat#181 (comment)

[vicuna]

Comment author: @nojb

At first sight it seems the difference is that ocamldep parses .ml files using Parse.use_file and the compiler uses Parse.implementation. For .mli files both use Parse.interface.

Anyone remembers the reason for the different handling of .ml files?

[vicuna]

Comment author: @nojb

#1766

[vicuna]

Comment author: @hhugo

commit 89077f5
Author: Damien Doligez <damien.doligez-inria.fr>
Date: Fri Aug 10 07:19:12 2012 +0000

Make ocamldep ignore toplevel directives in .ml files, as it did before
commit 12769.

[vicuna]

Comment author: @hhugo

GPR merged. This can be closed