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

cleanup some autotools legacy junk, fix most remaining warnings... #700

Closed

Conversation

edhartnett
Copy link
Contributor

@edhartnett edhartnett commented Nov 30, 2017

Minor cleanup of autotools to remove old stuff that is no longer is used and doesn't work.

Removed obsolete tst_h_filters.c (which had nothing to do with Dennis' recent filter work).

Also fixed most of the remaining warnings which occur with DAP enabled. These are easy fixes. Some tricky warnings are being tracked under their own tickets. Others I am not fixing due to complexity and lack of good understanding how to do so cleanly. (Suggestions welcome).

Fixes #699.
Fixes #698.
Fixes #697.
Fixes #696.
Fixes #694.
Fixes #662.

@edhartnett
Copy link
Contributor Author

I have just merged this into my other PR, so I will take it down shortly.

@edhartnett
Copy link
Contributor Author

I have merged this into PR #711. I will close this PR now.

@edhartnett edhartnett closed this Dec 4, 2017
@edhartnett edhartnett deleted the ejh_cleanup branch December 20, 2017 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment