+- Forum (https://www.boriel.com/forum)
+-- Forum: Compilers and Computer Languages (https://www.boriel.com/forum/forumdisplay.php?fid=12)
+--- Forum: ZX Basic Compiler (https://www.boriel.com/forum/forumdisplay.php?fid=11)
+---- Forum: Wishlist (https://www.boriel.com/forum/forumdisplay.php?fid=14)
+---- Thread: Add more library functions (/showthread.php?tid=289)
Add more library functions - britlion - 11-23-2010
Cool stuff.
I've added it as a code patch to the wiki library section. Are there any others in there, like LCD's proportional print routine or the nascent maths library section that you think are worth calling part of the include pack?
Re: Function call with string params bug (*solved*) - boriel - 11-24-2010
Yes, but like in this topic, we *must* intensively check them before including them into the library.
I've been quite busy these last months hence the delay, but I'm started to manage automatic testing, so things should go much faster from now onwards.
By the way, I think print64 is well tested enough, so I will include them in the library.
Also would like to include many other libraries (Mojon's codes in a /mj/ directory, and your fourspriter version among others) :wink:
BTW, there's a function called setattr(x, y, value) in ATTR.BAS library.
Programming an attrAddr function is also straighforward as this function is already defined in library-asm/attr.asm
I thing the routine in there is much faster an shorter (need more testing).
Re: Function call with string params bug (*solved*) - britlion - 11-24-2010
Thanks, Boriel.
Part of the duplication issue is lack of documentation - not knowing what's out there as a callable function.
Think that should be in library routines? I suppose knowing what's available in what include is going to help a lot
Re: Function call with string params bug (*solved*) - boriel - 11-24-2010
britlion Wrote:Thanks, Boriel.E.g. the print42 library is buggy. It contains the following code:
Part of the duplication issue is lack of documentation - not knowing what's out there as a callable function.
Think that should be in library routines? I suppose knowing what's available in what include is going to help a lot
Code:
newline:
LD DE,(63536)
CALL nxtline ; move to next line
LD (63536),DE ; and go on to next character
JR nextCharRe: Add more library functions - britlion - 11-24-2010
I completely agree I need other people to test these. I didn't write the original code, just reworked it from well known sources into shape; which means there are likely to be bugs.
I'll look into this one after work today.
Re: Add more library functions - boriel - 11-24-2010
britlion Wrote:I completely agree I need other people to test these. I didn't write the original code, just reworked it from well known sources into shape; which means there are likely to be bugs.Ok, I will later. :wink:
I'll look into this one after work today.
Ah, I forgot: please, put the URL or whatever of the original code (based upon: bla bla bla). Not only gives them credit, but also we can blame on them in case of a bug, like this! :mrgreen:
Re: Add more library functions - boriel - 11-24-2010
britlion Wrote:I completely agree I need other people to test these. I didn't write the original code, just reworked it from well known sources into shape; which means there are likely to be bugs.I've made some changes to your routine. Please download and compare.
I'll look into this one after work today.
Some corrections:
- Removed the LD DE,(XXXX) block. It's useless and might corrupt the HEAP
- Check if HL==0 (NULL) at the beginning of the asm code, and return if so (this can happen if not enough memory)
- Remove the FASTCALL and, instead of returning immediately, jump to the end of subroutine whenever required. The string ByVal parameters must be freed upon return, or there will be a memory leak (yes, the same that happened with print64
)
- Make as much labels as possible LOCAL, so it won't pollute namespace and won't collide with other routines. E.g. nextchar label could be used in many other routines (e.g. print64).