On Tue, 13 Apr 2004, Arkady V.Belousov wrote:

> 13-сав-2004 11:54 [EMAIL PROTECTED] (Bart Oldeman) wrote to
> [EMAIL PROTECTED]:
>
> > +++ task.c    13 Apr 2004 11:54:09 -0000      1.41
> > +  fstrcpy(Shell + strlen(Shell), MK_FP(FP_SEG(Config), Config->cfgInitTail));
> >    endp =  Shell + strlen(Shell);
>
>   fstrcpy(endp = Shell + strlen(Shell),
>           MK_FP(FP_SEG(Config), Config->cfgInitTail));

this won't work. We need the strlen of Shell after the fstrcpy. This is
really an "fstrcat" except that's not in asmsupt.asm.

endp = Shell + strlen(Shell) + strlen(Config->cfgInitTail);
fstrcpy(Shell + strlen(Shell), MK_FP(FP_SEG(Config), Config->cfgInitTail));
would work but I don't see the point.

> >  STATIC VOID InitPgm(BYTE * pLine)
> >  {
> > +  static char init[NAMEMAX];
> > +  static char inittail[NAMEMAX];
> > +
> > +  Config.cfgInit = init;
> > +  Config.cfgInitTail = inittail;
>
>      As I understand, these assignments may be performed statically.
> Especially, if cfgInit and cfgInitTail fields will be moved out from Config
> structure.

??? I don't understand what you mean here.

> > +             mov     cx,-2 + init_end wrt INIT_TEXT     ; word aligned
>
>      BTW, does (or not) NASM supports TASM/MASM compatible syntax:
>
>          mov     cx,offset INIT_TEXT:init_end - 2       ; word aligned

NASM doesn't support "offset".

Bart



-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id70&alloc_id638&op=click
_______________________________________________
Freedos-kernel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/freedos-kernel

Reply via email to