Simplify the conditions at the beginning of the theme

nordtheme / vim

An arctic, north-bluish clean and elegant Vim theme.

https://www.nordtheme.com/ports/vim

MIT License

2.51k stars 275 forks source link

Simplify the conditions at the beginning of the theme #295

Closed jvoisin closed 2 years ago

jvoisin commented 2 years ago

Use ternary when possible
use get instead of nested if
Inline the s:logWarning function since it was only used once
Group some block where it made sense

svengreb commented 2 years ago

Hi @jvoisin :wave:, thanks for your contribution :+1: I'm currently busy with the preparations for the roadmap and future plan of the whole Nord project, but I'll try to review the changes within the next week(s).