diff options
author | Su Hang <suhang16@mails.ucas.ac.cn> | 2018-02-25 12:35:58 +0800 |
---|---|---|
committer | Stefan Hajnoczi <stefanha@redhat.com> | 2018-03-05 09:03:17 +0000 |
commit | a1515161b563acd0986867fc37084c1cf952ba35 (patch) | |
tree | 2883fccdb584c09d82694716fafc929f6482b587 /CODING_STYLE | |
parent | 42fa27253cfa8b9ad7d41ce63a733a4c623ffc74 (diff) |
util/uri.c: wrap single statement blocks with braces {}
For this patch, using curly braces to wrap `if` `while` `else` statements,
which only hold single statement. For example:
'''
if (cond)
statement;
'''
to
'''
if (cond) {
statement;
}
'''
And using tricks that compare the disassemblies before and after
code changes, to make sure code logic isn't changed:
'''
git checkout master
make util/uri.o
strip util/uri.o
objdump -Drx util/uri.o > /tmp/uri-master.txt
git checkout cleanupbranch
make util/uri.o
strip util/uri.o
objdump -Drx util/uri.o > /tmp/uri-cleanup.txt
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'CODING_STYLE')
0 files changed, 0 insertions, 0 deletions