edilbert / bsa Goto Github PK

Hi! Can you add some statement about the license under which you're distributing this? For example MIT, BSD, GPL...
Thanks.

Danke. Sehr interessant.
Warum man >2000 Zeilen C Code schreiben kann, aber keine 20 Zeilen readme.txt entzieht sich allerdings meinem Verständnis :-(
Das am Anfang des C Sourcecodes vieles steht ist gut!

Aber gerade .STORE ist da angegeben mit:
.STORE BASIC_ROM,$2000,"basic.rom" write binary image file "basic.rom"

Aber erst nach dem Studium des Quelltextes, kann ein C Sourcecode Leser sehen:
char *ParseStoreData(char *p)
{
int Start,Length,i;
char Filename[80];

if (Phase < 2) return p;
p = EvalOperand(p,&Start,0);
if (Start < 0 || Start > 0xffff)
{
ErrorMsg("Illegal start address for STORE %d\n",Start);
exit(1);
}
p = NeedChar(p,',');
if (!p)
{
ErrorMsg("Missing ',' after start address\n");
exit(1);
}
p = EvalOperand(p+1,&Length,0);
if (Length < 0 || Length > 0x10000)
{
ErrorMsg("Illegal length for STORE %d\n",Length);
exit(1);
}
p = NeedChar(p,',');
if (!p)
{
ErrorMsg("Missing ',' after length\n");
exit(1);
}
p = NeedChar(p+1,'"');
if (!p)
{
ErrorMsg("Missing quote for filename\n");
exit(1);
}

=> d.h.:
;.STORE ,,"filename"
.STORE $280,$100,"out.bin"

Dabei MUSS man eine Länge angeben - für Eproms sicher ok. Für raw binare aber blöd.
Hier fehlt ein Parameter, der als Platzhalter für die Filelänge dient!

Genauso die Startadresse - kann mal doch aus der ORG / * = Anweisung nehmen..
(z.B. -1 für beide = Start aus org/*= oder 0 und Länge = Binärlänge)

Noch ein weiterer Punkt. Source muss zwingend .asm heißen.
Stört nicht, muss man aber wissen und finde ich blöd.

Ansonsten schöner Assembler!

LG Peter

hi,

i am trying to compile BSA for windows (using Pelles C development kit for Desktop Windows (it is free)).

I have trouble because of strncasecmp / strcasecmp.

Doing a quick research, it is said, they are defined in the libc standard library in BSD/POSIX but they are "non standard" (ansi c?).

I was able to compile it with

#ifdef _WIN32
#define strcasecmp _stricmp
#define strncasecmp _strnicmp
#endif

Just found a compile option "Define compatibility names" which also works. If someone is interesting, screenshot:

• = $fffd

.byte 1
.byte 2
.byte 3
.byte 4

The reason should be line 3332:

Src = MallocOrDie(strlen(argv[ic] + 4 + 1));

I guess you mean

Src = MallocOrDie(strlen(argv[ic]) + 4 + 1);

With this fix, BSA is working much better on windows.