build: remove HAVE_LIMITS_H check #2215

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Owner

jay commented Jan 3, 2018

.. because limits.h presence isn't optional, it's required by C89.

Ref: http://port70.net/~nsz/c/c89/c89-draft.html#2.2.4.2

Closes #xxxx

build: remove HAVE_LIMITS_H check
.. because limits.h presence isn't optional, it's required by C89.

Ref: http://port70.net/~nsz/c/c89/c89-draft.html#2.2.4.2

Closes #xxxx

@jay jay added the build label Jan 3, 2018

@jay jay requested review from bagder and gvanem Jan 3, 2018

bagder approved these changes Jan 4, 2018

I'm sure we once had a reason for this, but that was a long time ago so let's try to do the right thing now and see how it plays out...

@jay jay closed this in 908a9a6 Jan 6, 2018

@jay jay deleted the jay:remove_HAVE_LIMITS_H branch Jan 7, 2018

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