Support for immediate command parsing was implemented in anticipation of the temporary console mode interface, essentially the way GW-Basic works. However, now that the project is being transitioned to Qt for a GUI, these immediate commands are no longer needed. Another factor for removing this was that a lot of modifications to the command parsing functions, which accounted for about a third of the Parser code, would need to be converted to using Qt functions. A waste of time since this would eventually be removed. Therefore, immediate commands support was removed along with parser test 1.
The only code currently using the String class was the Token class. The Parser class generated tokens and therefore the strings inside the tokens. The String class has been removed from the Parser and Token classes, which now uses Qt functions for parsing and no longer uses any of the standard C library functions (the goal for the entire program, but one step at a time).
Since the String class is no longer used, the header and source files for this class were also removed along with the string test program and expected results file. The other three test programs were also removed because these no longer apply, which were for testing conversion of number strings (Qt functions now used), exceptions thrown from constructors (no longer used), and testing operator processing on a stack (Stack class was already removed).
Upon testing on Windows, the range error checking problem reported on October 14 returned. This was due to how numbers are converted in the Qt libraries. Originally on Linux exponents -308 and below caused a range error, but on Windows exponents did cause a range error until -324. Now with Qt, on Linux exponents -324 and below cause a range error and on Windows it takes exponents -509 and below. Therefore, the test value was changed to 1.234e-509.
[commit 6b4afaa538] [commit 348339c7a7] [commit 819fac5185]
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.)