Before moving on how to build with CMake, A problem was discovered on Windows while writing the build procedure. Specifically with the test_codes.awk script that generates the test_codes.h header file (containing the strings of the code names used in the parser test code). This is the same problem corrected with the winfix0.1.15 branch off of the master branch.
The same fix can't be used for Linux as the script no longer works on Linux. The issue is that when git checks out files are Windows, all the source files are automatically converted to DOS (CRLF) format while on Linux, they are left in Unix (newline) format. Because the table.cpp source file (the source of the code names) is in DOS format on Windows, there is an extra CR at the end of each line, which is treated as a normal character. The way this was fixed in the winfix0.1.15 branch was to set the record separator (RS) variable to "\r\n" in the awk script so that CRLF characters are properly handled as line separators.
However, setting RS to "\r\n" on Linux breaks the awk script because now it sees a single huge line since there are no "\r\n" to be interpreted as line separators in the table.cpp source file that is in Unix format. Currently the script was subtracting 5 from the length of the field containing the "Xxx_Code" characters, removing the "_Code" part to get the "Xxx" part. With the extra CR character, the "_" was left on the string leaving "Xxx_" instead.
The length function was changed to the index function to search for the position of the "_Code" part. One was subtracted from this value to calculate the length of the "Xxx" part. This change works with both DOS and Unix format files. This change was committed and pushed to the github repository. Next up, how to obtain this latest commit for an already cloned repository...
Subscribe to:
Post Comments (Atom)
No comments:
Post a Comment
All comments and feedback welcomed, whether positive or negative.
(Anonymous comments are allowed, but comments with URL links or unrelated comments will be removed.)