Paul A Bristow wrote:
| rationale.html. cubeRootFour is called two_pow_two_thirds in the | documentation.
Sorry I have not updated this much after the discussion during review. I'll do some tidying up - thanks for noting this.
Another typo: sqrt32 is commented as sqrt(10) in the source. And a question: Sometimes, you omit the trailing 0(s) from some constants. Could you change the scripts to not do this? I could make use of it, the current version of my library (not yet published) includes a "port" of your constants, that is also capable of retrieving the original value (that is, the string in the source!) and pass it on to generic mappers to initialize e.g. RWDecimal<RWMP3Int> or other types with a ctor taking const char*. Having the trailing zeros would make a better statement about the real value-range it represents. Example: 1.23 specifies a number from the range [1.225-1.235) while 1.230 specifies a number from the range [1.2295-1.2305) and thus the latter yields more information. BTW: In case you don't like me to port your header, let me know. Of course, credit will be given if you don't have any objections against the port. Regards, Daniel