From 844382be19860ca2803ec750c55d24886fc21622 Mon Sep 17 00:00:00 2001 From: "e.gavrin" Date: Tue, 22 Jul 2014 21:53:18 +0400 Subject: [PATCH] Moved scopes of several variables closer to usage; --- src/libjsparser/lexer.c | 9 +++++++-- src/libjsparser/parser.c | 2 +- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/src/libjsparser/lexer.c b/src/libjsparser/lexer.c index 9d06ab772..2d0433c91 100644 --- a/src/libjsparser/lexer.c +++ b/src/libjsparser/lexer.c @@ -253,11 +253,14 @@ add_num_to_seen_tokens (num_and_token nat) uint8_t lexer_get_strings (const char **strings) { - int i; - if (strings) + { + int i; for (i = 0; i < seen_names_count; i++) + { strings[i] = seen_names[i].str; + } + } return seen_names_count; } @@ -276,7 +279,9 @@ lexer_get_nums (int *nums) int i; for (i = 0; i < seen_nums_count; i++) + { nums[i] = seen_nums[i].num; + } return seen_nums_count; } diff --git a/src/libjsparser/parser.c b/src/libjsparser/parser.c index 2e82c59b4..cd1e50257 100644 --- a/src/libjsparser/parser.c +++ b/src/libjsparser/parser.c @@ -319,7 +319,6 @@ parse_argument_list (argument_list_type alt, T_IDX obj) lhs = 0, args[3+1/* +1 for stack protector */], current_arg = 0; - bool is_first_opcode = true; switch (alt) { @@ -361,6 +360,7 @@ parse_argument_list (argument_list_type alt, T_IDX obj) skip_newlines (); if (tok.type != close_tt) { + bool is_first_opcode = true; while (true) { if (is_first_opcode)