Skip to content
This repository has been archived by the owner on May 4, 2018. It is now read-only.

process: remove debug perror() prints #1157

Closed
wants to merge 1 commit into from
Closed

process: remove debug perror() prints #1157

wants to merge 1 commit into from

Conversation

indutny
Copy link
Contributor

@indutny indutny commented Feb 26, 2014

fix #1128

@Nodejs-Jenkins
Copy link

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

The following commiters were not found in the CLA:

  • Fedor Indutny

You can fix all these things without opening another issue.

Please see CONTRIBUTING.md for more information

@saghul
Copy link
Contributor

saghul commented Mar 3, 2014

LGTM. I wonder if we should still have them in debug builds?

@indutny
Copy link
Contributor Author

indutny commented Mar 3, 2014

@saghul I think no, we are returning error as an exit code, so it should be fine anyway.

@sam-github
Copy link

Thanks, Fedor.

@indutny
Copy link
Contributor Author

indutny commented Mar 3, 2014

Landed in c0c9480, thank you!

@indutny indutny closed this Mar 3, 2014
@indutny indutny deleted the fix/gh-1128 branch March 3, 2014 17:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants