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.

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

Nit-picking #50

Closed
RenniePet opened this issue Jan 26, 2017 · 5 comments
Closed

Nit-picking #50

RenniePet opened this issue Jan 26, 2017 · 5 comments

Comments

@RenniePet
Copy link

Looking at the source code, there's a tiny inconsistency between what the guard statement on line 938 does and what the following error message says.

(Are you interested in receiving this kind of information?)

@RenniePet
Copy link
Author

And while I'm at it ...

Line 398:
/// Returns a string description of the information.

@RenniePet
Copy link
Author

And the inconsistency I noted in my first message is repeated four other places.

@billabt
Copy link
Collaborator

billabt commented Jan 26, 2017

Absolutely. I like clean, correct and concise code and comments. The more the merrier. 😀

@billabt
Copy link
Collaborator

billabt commented Jan 27, 2017

I've corrected the nits you've found previously. Feel free to send more my way. Thanks.

@billabt billabt closed this as completed Jan 27, 2017
@RenniePet
Copy link
Author

Glad you could use my input. If I find more I'll let you know.

And now for something completely different - see Stack Overflow :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants