RE: [PATCH 2/4] Handle the conversion failure when parsing item

Hi Denis, Thank you, and I will pay attention to this in future. I'm always wondering what's the complete coding style we should follow. Can we have some document to describe them? I have collected some of them: 1. Check the patch using checkpatch.pl (checkpatch.pl --no-tree patch_name). In theory, you need to clean up all the warnings and errors except one "ERROR: Missing Signed-off-by: line(s)". However, sometimes, the warning of exceeding maximum characters for a line (80 characters) can be ignored. 2. There should be a blank line before if statement, unless it is nested and not preceded by an expression or variable declaration. 3. There should be space before and after operator. 4. It's better not to have a complicated statement for if. You may judge its contrary condition and return | break | continue ASAP. 5. Better to use abbreviation, rather than full name, to name a variable, function, struct, etc. Any others? Regards, -Yang