search

ooc-lang / rock

:ocean: self-hosted ooc compiler that generates c99
http://ooc-lang.org/
MIT License
404 stars 40 forks source link

Add -pipe to the list of flags rock directly recognizes. #913

Closed ibara closed 9 years ago

ibara commented 9 years ago

While here, Flags.ooc has MSDOS line endings. Convert to Unix line endings.

As it says on the tin. Rationale: OpenBSD (and I think some if not all of the BSDs) default to CFLAGS '-O2 -pipe' This way rock won't complain it doesn't understand -pipe and force us to do '-O2 -pipe +-pipe'

ibara commented 9 years ago

To add: I chose to leave -pipe out of the manual page because the -O levels are not in the man page, and I see -pipe in the same vein.

Can add either/both to the manual page if that's desirable.

fasterthanlime commented 9 years ago

@ibara I think all options should be documented in the man page, I believe it got a bit out of sync with rock's help output (which, in turn, is not always the most up-to-date). But a man page update is slated for 0.9.11 anyway, so there's no rush.

ibara commented 9 years ago

I can at least add -O and -pipe to the manual page.

fasterthanlime commented 9 years ago

Sure thing :)

ibara commented 9 years ago

Done!

fasterthanlime commented 9 years ago

Merged! (by cherry-picking commits).