useDecimal comment should state true?

mkxml / vscode-filesize

filesize package for Visual Studio Code

MIT License

60 stars 13 forks source link

Closed matthewfarlymn closed 6 years ago

matthewfarlymn commented 6 years ago

The current comment uses 'false' twice.

// Defaults to false, using IEC's representation. Set to false to get SI representation.
  "filesize.useDecimal": false

Should it not state 'true' for the second sentence?

// Defaults to false, using IEC's representation. Set to true to get SI representation.
  "filesize.useDecimal": false

mkxml commented 6 years ago

@matthewfarlymn Good catch! You're correct.

This config used to be reversed before and I forgot to change this part of the comment.

If you open a PR with this change I'll happily merge it.

Thanks!

mkxml commented 6 years ago

Done in #13.

matthewfarlymn commented 6 years ago

Thanks :)