> 1. About 'const _print_max_var_size_dec' in the > print_scalable library: When user may declare this, it > should not have a name starting with an underscore.
I wasn't sure, and I thought, better use an underscore wrong than omit it where it should be... > Can we drop the '_dec' of this name? Decimal is the > compiler default for numbers, so why the emphasis on 'dec' > making the name longer than needed? Everything apart from dec is not scalable at all, so I named it like that. I wouldn't object in neither dropping the "dec" part nor the starting "_" of the name. > 2. print_bit_10() is not an intuitive name and not > consistent with the other names: print_bit_binary() seems > better to me. > I also would like to suggest to rename > print_bit_truefalse() to print_bit_logic(), and > print_bit_highlow() to print_bit_level(). > BTW: the suffix '_binary' is not consistent with '_hex' and > '_dec', so it better be renamed to '_bin'. I agree! I just wanted to keep compatibility to conventional print.jal. > 4. Not a single procedure is documented. Err, well... Can I say here it is for compatibility reasons? :-D Greets, Kiste -- You received this message because you are subscribed to the Google Groups "jallib" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/jallib?hl=en.
