> Added:
>     trunk/unvalidated/include/external/temperature/tc77.jal
>     trunk/unvalidated/sample/by_device/16f876a/test_tc77.jal

Look nice ! Can we add a "temp_" to the filename ("temp_tc77.jal) or
the like ? It'll help users know what this is about (remember, libs
will be flatten while releasing, so dirname won't help), and be more
consistent with current existing libs (prefixed with pwm_, adc_,
serial_, i2c_, lcd_). I know this is about external part, and
googleing "tc77" will get the answers, it's just to be make users'
life easier. What do you think about this ?

Another point. There's a "nasty" indentation in your file: procedures'
body aren't indented.

Anyone managed to get "indent" program working with jal ?

OK, I'm done criticizing ! :) Thanks for this commit.

Cheers,
Seb
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to