Added checkPortStatuses method #5

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@nickdaugherty

Checks the status of all ports in a range, returning an array of port / status pairs once complete

@baalexander
Owner

Thanks for the feature @nickdaugherty.

Do you happen to know why the diff is so large for the one function?

@nickdaugherty

No idea why the diff is wrong, I noticed it too. It was like that locally
before I pushed to github, so must have been something in my machine.
On Jul 29, 2012 9:08 PM, "Brandon Alexander" <
reply@reply.github.com>
wrote:

Thanks for the feature @nickdaugherty.

Do you happen to know why the diff is so large
for the one function?


Reply to this email directly or view it on GitHub:
#5 (comment)

@EndangeredMassa
Contributor

It looks like this was a line ending issue. You can use the file command to check.

When I use file on the original files, I get ASCII text. When I use it on one of the files in this PR, I get ASCII text, with CRLF line terminators.

Look at your editor's settings for something that lets you toggle between windows-style and unix-style line endings. Consider changing it to unix-style for this project, then update this PR.

@EndangeredMassa
Contributor

I'm closing this old PR. If you would still like to pursue it, please reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment