Skip to content
This repository was archived by the owner on Jun 15, 2023. It is now read-only.

Commit 8bf2e17

Browse files
author
Iwan
committed
Cleanup dead code and fix exception reports.
1 parent 7b1b39e commit 8bf2e17

File tree

7 files changed

+16
-18
lines changed

7 files changed

+16
-18
lines changed

Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,17 +71,17 @@ lib/test.exe: $(TEST_FILES)
7171
$(OCAMLOPT) $(OCAMLFLAGS) -O2 -o ./lib/test.exe -bin-annot -I +compiler-libs ocamlcommon.cmxa -I src -I tests $(TEST_FILES)
7272

7373
test: build-native lib/test.exe
74-
./node_modules/.bin/reanalyze -all-cmt . -suppress tests
74+
./node_modules/.bin/reanalyze -all-cmt . -suppress tests -suppress benchmarks
7575
./node_modules/.bin/jest
7676
./lib/test.exe
7777

7878
roundtrip-test: bootstrap lib/test.exe
79-
./node_modules/.bin/reanalyze -all-cmt . -suppress tests
79+
./node_modules/.bin/reanalyze -all-cmt . -suppress tests -suppress benchmarks
8080
ROUNDTRIP_TEST=1 ./node_modules/.bin/jest
8181
./lib/test.exe
8282

8383
reanalyze: build-native lib/test.exe
84-
./node_modules/.bin/reanalyze -all-cmt . -suppress tests
84+
./node_modules/.bin/reanalyze -all-cmt . -suppress tests -suppress benchmarks
8585

8686
clean:
8787
rm -rf src/*.cm*

src/res_cli.ml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ module CliArgProcessor = struct
270270
end
271271

272272

273-
let () =
273+
let [@raises exit] () =
274274
if not !Sys.interactive then begin
275275
ResClflags.parse ();
276276
match !ResClflags.files with
@@ -294,6 +294,4 @@ let () =
294294
~report:!ResClflags.report
295295
filename
296296
) files
297-
298-
end
299-
[@@raises exit]
297+
end

src/res_core.ml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5710,8 +5710,16 @@ and parseWithConstraint p =
57105710
)
57115711
end
57125712
| token ->
5713+
(* TODO: implement recovery strategy *)
57135714
Parser.err p (Diagnostics.unexpected token p.breadcrumbs);
5714-
exit (-1) [@doesNotRaise] (* TODO: handle this case *)
5715+
Parsetree.Pwith_type (
5716+
(Location.mknoloc (Longident.Lident "")),
5717+
Ast_helper.Type.mk
5718+
~params:[]
5719+
~manifest:(Recover.defaultType ())
5720+
~cstrs:[]
5721+
(Location.mknoloc "")
5722+
)
57155723

57165724
and parseModuleTypeOf p =
57175725
let startPos = p.Parser.startPos in

src/res_parser.ml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -160,5 +160,3 @@ let lookahead p callback =
160160
p.comments <- comments;
161161

162162
res
163-
164-
let comments p = Array.of_list (List.rev p.comments)

src/res_parser.mli

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,5 +47,3 @@ val checkProgress:
4747
result: 'a ->
4848
t ->
4949
'a option
50-
51-
val comments: t -> Comment.t array

src/res_scanner.ml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ module CharacterCodes = Res_character_codes
22
module Diagnostics = Res_diagnostics
33
module Token = Res_token
44
module Comment = Res_comment
5-
type mode = Template | Jsx | Diamond
5+
type mode = Jsx | Diamond
66

77
type t = {
88
filename: string;
@@ -23,9 +23,6 @@ type t = {
2323
let setDiamondMode scanner =
2424
scanner.mode <- Diamond::scanner.mode
2525

26-
let setTemplateMode scanner =
27-
scanner.mode <- Template::scanner.mode
28-
2926
let setJsxMode scanner =
3027
scanner.mode <- Jsx::scanner.mode
3128

src/res_scanner.mli

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
type mode = Template | Jsx | Diamond
1+
type mode = Jsx | Diamond
22

33
type t = {
44
filename: string;
@@ -23,7 +23,6 @@ val scan: t -> (Lexing.position * Lexing.position * Res_token.t)
2323

2424
val isBinaryOp: bytes -> int -> int -> bool
2525

26-
val setTemplateMode: t -> unit
2726
val setJsxMode: t -> unit
2827
val setDiamondMode: t -> unit
2928
val popMode: t -> mode -> unit

0 commit comments

Comments
 (0)