Refactored opts parsing.

src/erlang_v8_vm.erl

@@ -196,19 +196,18 @@ priv_dir() ->
 
 %% @doc Parse proplists/opts and populate a state record.
 parse_opts(Opts) ->
-    parse_opts(Opts, #state{initial_source = []}).
+    lists:foldl(fun parse_opt/2, #state{initial_source = []}, Opts).
 
-parse_opts([], State) ->
-    State;
+%% @doc Append source specified in source option.
+parse_opt({source, S}, #state{initial_source = InitialSource} = State) ->
+    State#state{initial_source = [S|InitialSource]};
 
-parse_opts([{source, S}|T], #state{initial_source = InitialSource} = State) ->
-    parse_opts(T, State#state{initial_source = [S|InitialSource]});
-
-parse_opts([{file, F}|T], #state{initial_source = InitialSource} = State) ->
+%% @doc Read contents of file option and append to state.
+parse_opt({file, F}, #state{initial_source = InitialSource} = State) ->
     %% Files should probably be read in the OS process instead to prevent
     %% keeping multiple copies of the JS source code in memory.
     {ok, S} = file:read_file(F),
-    parse_opts(T, State#state{initial_source = [S|InitialSource]});
+    State#state{initial_source = [S|InitialSource]};
 
-parse_opts([_|T], State) ->
-    parse_opts(T, State).
+%% @doc Ignore unknown options.
+parse_opt(_, State) -> State.