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

Argument validation #1

Merged
merged 1 commit into from Jun 2, 2016
Merged

Argument validation #1

merged 1 commit into from Jun 2, 2016

Conversation

dacamp
Copy link
Contributor

@dacamp dacamp commented Jun 1, 2016

The panic was a little jarring, so I added a simple validator.

$ pxl
panic: open pxl: no such file or directory

goroutine 1 [running]:
panic(0x16c3e0, 0xc82007a330)
    /usr/local/Cellar/go/1.6.2/libexec/src/runtime/panic.go:481 +0x3e6
main.main()
    /Users/dcam/gopath/src/github.com/ichinaski/pxl/main.go:50 +0xb2
$ ./pxl
Usage: ./pxl <filename>

Close the image with <ESC>.

@ichinaski
Copy link
Owner

Looks good!

@ichinaski ichinaski merged commit 52357dd into ichinaski:master Jun 2, 2016
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

Successfully merging this pull request may close these issues.

None yet

2 participants