Linux Pollution Cleanup

Reporter: RobertBColton  |  Status: open  |  Last Modified: November 20, 2018, 07:48:52 PM
This pull request is a friend of #1445 that attempts to resolve #1327 and add a test to prevent it from coming back. I basically just moved the includes around so that <X11/Xlib.h> is not included in SHELLmain and only the headers with user functions are. This is why XLIBmain.h and XLIBwindow.h are not included in Platforms/xlib/include.h anymore, because they only have implementation stuff, no actual user functions declared in them. Most platform user functions have their declarations generalized now anyway.

My change to the math test produced this in the test harness job of the first commit of this pullrequest:

Line 213, position 6 (absolute 8474): Invalid assignment to function `time'

That means the test successfully reproduced the bug in #1327 and this pull request fixes the issue if it passes the test at the time this is merged.

Please sign in to post comments, or you can view this issue on GitHub.