From 941f46038415bfdc52f61b68511c83fed59fc2fe Mon Sep 17 00:00:00 2001 From: Fabio Alessandrelli Date: Mon, 3 Oct 2016 02:40:53 +0200 Subject: [PATCH] Properly encode InputEvent of type NONE. Fix #5987 In the editor settings you can disable default editor shortcuts. When a default shortcut is disabled an InputEvent of type NONE must be stored in the config file to allow the editor to remember that setting. variant_parser.cpp was not properly encoding InputEvent of type NONE causing the "corruption" of the editor settings file. --- core/variant_parser.cpp | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/core/variant_parser.cpp b/core/variant_parser.cpp index 023605a952e..6b3828a5726 100644 --- a/core/variant_parser.cpp +++ b/core/variant_parser.cpp @@ -986,7 +986,18 @@ Error VariantParser::parse_value(Token& token,Variant &value,Stream *p_stream,in InputEvent ie; - if (id=="KEY") { + if (id=="NONE") { + + ie.type=InputEvent::NONE; + + get_token(p_stream,token,line,r_err_str); + + if (token.type!=TK_PARENTHESIS_CLOSE) { + r_err_str="Expected ')'"; + return ERR_PARSE_ERROR; + } + + } else if (id=="KEY") { get_token(p_stream,token,line,r_err_str); if (token.type!=TK_COMMA) { @@ -2093,6 +2104,9 @@ Error VariantWriter::write(const Variant& p_variant, StoreStringFunc p_store_str case InputEvent::JOYSTICK_MOTION: { str+="JAXIS,"+itos(ev.joy_motion.axis)+","+itos(ev.joy_motion.axis_value); } break; + case InputEvent::NONE: { + str+="NONE"; + } break; default: {} }