-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Cleanup event-config.h #1645
base: master
Are you sure you want to change the base?
Cleanup event-config.h #1645
Conversation
@@ -427,10 +390,6 @@ | |||
/* Define if timerclear is defined in <sys/time.h> */ | |||
#cmakedefine EVENT__HAVE_TIMERCLEAR 1 | |||
|
|||
/* Define if timercmp is defined in <sys/time.h> */ | |||
#cmakedefine EVENT__HAVE_TIMERCMP 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes like this should be complete, i.e removing the check for timercmp
from CMake, and doing the corresponding changes in the Autotools build system at the same time. Otherwise you will (likely) end up with divergence in behaviour between the two systems.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Coeur let's sync the configure.ac as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like that: ccb0c2b ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct, but this is not all, you need to adjust the configure.ac for other changes, i.e. remove HAVE_TAILQFOREACH, and likely something else, I did not look deeply enough for now, please take a look
8d38de5
to
ccb0c2b
Compare
Fix #1332.
The duplication was introduced with e415196
Also addresses partially #1329, but I only removed the stuff for which I was 100% sure it was unused.