Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

simplify the if/else logic #179

Merged
merged 13 commits into from
Jan 25, 2019
Merged

simplify the if/else logic #179

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
1371618
fix typo
lewgun Jan 10, 2019
98e20f6
remove the redundant parentheses
lewgun Jan 10, 2019
65f0c70
Merge branch 'master' of https://github.com/lewgun/tidb-tools
lewgun Jan 10, 2019
bf55d1f
adjust a few error messages
lewgun Jan 10, 2019
68cec5b
Merge pull request #1 from pingcap/master
lewgun Jan 15, 2019
77c56d7
adjust a few error messages
lewgun Jan 15, 2019
4797aa1
Merge pull request #2 from pingcap/master
lewgun Jan 15, 2019
7bba47d
fix typos &
lewgun Jan 15, 2019
df4ff93
Merge pull request #3 from pingcap/master
lewgun Jan 16, 2019
a49ecbf
simplify the if/else logic
lewgun Jan 16, 2019
758258e
add the missing 'break'
lewgun Jan 16, 2019
ca09607
make the golint happy
lewgun Jan 20, 2019
923c7b8
compare the two strings' length first
lewgun Jan 25, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge pull request #2 from pingcap/master 
sync with master
  • Loading branch information
@lewgun
lewgun authored Jan 15, 2019
commit 4797aa1c3368bf9590ca7476a66fb1e53f253762

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.