search

rescript-association / reanalyze

Experimental analyses for ReScript and OCaml: globally dead values/types, exception analysis, and termination analysis.
MIT License
277 stars 20 forks source link

Support ocaml 5.2 #198

Closed cristianoc closed 3 months ago

cristianoc commented 3 months ago

Fixes https://github.com/rescript-association/reanalyze/issues/196

cknitt commented 3 months ago

@cristianoc Thanks! Unfortunately I now get the errors

File "src/Noalloc.ml", line 67, characters 4-20:
Error: The constructor Tpat_var expects 3 argument(s),
       but is applied here to 2 argument(s)
File "src/Arnold.ml", line 518, characters 41-56:
Error: The constructor Tpat_var expects 3 argument(s),
       but is applied here to 2 argument(s)
File "src/DeadCommon.ml", line 339, characters 8-48:
Error: The constructor Tpat_var expects 3 argument(s),
       but is applied here to 2 argument(s)
File "src/Exception.ml", line 481, characters 6-40:
Error: The constructor Tpat_var expects 3 argument(s),
       but is applied here to 2 argument(s)
cristianoc commented 3 months ago

@cknitt check again now

cknitt commented 3 months ago

Thanks! And sorry, we pushed fixes more or less simultaneously. 5.2 CI actually does work once the .opam file is updated.