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

Completed Implementation of Cat #54

Merged
merged 1 commit into from Apr 11, 2016
Merged

Completed Implementation of Cat #54

merged 1 commit into from Apr 11, 2016

Conversation

mmstick
Copy link
Contributor

@mmstick mmstick commented Apr 11, 2016

  • Added exit_status to the Program structure to keep track of errors.
  • Removed the vector for buffering the file and instead opted for printing to stdout directly.
  • Implemented caret (^) and M- notation for showing non-printable characters.

@ticki
Copy link
Contributor

ticki commented Apr 11, 2016

It still fails.

@mmstick
Copy link
Contributor Author

mmstick commented Apr 11, 2016

That's because it's not this code that is causing the error. Recent commits added to master are causing all builds to fail, namely that a MAN_PAGE constant was added to many commands but those commands aren't using the constant in the code.

@ticki
Copy link
Contributor

ticki commented Apr 11, 2016

Oh, right. The master is broken.

@ticki
Copy link
Contributor

ticki commented Apr 11, 2016

:shipit:

@ticki ticki merged commit 9d1ddd7 into redox-os:master Apr 11, 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
2 participants