Hugh McMaster : regedit: Remove un-necessary parser state assignment if we are returning to the same state.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: c750c2a98d20ed5bb91ccbe380fd9fb44975f832
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=c750c2a98d20ed5bb91ccbe380...

Author: Hugh McMaster <hugh.mcmaster(a)outlook.com>
Date:   Fri Jun 30 13:42:43 2017 +0000

regedit: Remove un-necessary parser state assignment if we are returning to the same state.

Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 programs/regedit/regproc.c | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/programs/regedit/regproc.c b/programs/regedit/regproc.c
index d6ea5c7..e5a4f74 100644
--- a/programs/regedit/regproc.c
+++ b/programs/regedit/regproc.c
@@ -532,7 +532,7 @@ static WCHAR *parse_win31_line_state(struct parser *parser, WCHAR *pos)
         return NULL;
 
     if (strncmpW(line, hkcr, ARRAY_SIZE(hkcr)))
-        goto invalid;
+        return line;
 
     /* get key name */
     while (line[key_end] && !isspaceW(line[key_end])) key_end++;
@@ -548,7 +548,7 @@ static WCHAR *parse_win31_line_state(struct parser *parser, WCHAR *pos)
     if (open_key(parser, line) != ERROR_SUCCESS)
     {
         output_message(STRING_OPEN_KEY_FAILED, line);
-        goto invalid;
+        return line;
     }
 
     parser->value_name = NULL;
@@ -558,10 +558,6 @@ static WCHAR *parse_win31_line_state(struct parser *parser, WCHAR *pos)
 
     set_state(parser, SET_VALUE);
     return value;
-
-invalid:
-    set_state(parser, PARSE_WIN31_LINE);
-    return line;
 }
 
 /* handler for parser LINE_START state */
@@ -589,7 +585,6 @@ static WCHAR *line_start_state(struct parser *parser, WCHAR *pos)
         case '\t':
             break;
         default:
-            set_state(parser, LINE_START);
             return p;
         }
     }