Hi!
11-Сен-2004 15:52 [EMAIL PROTECTED] (Luchezar Georgiev) wrote to
[EMAIL PROTECTED]:
LG> Log Message:
LG> break.c: check_handle_break() return type was undefined if no Ctrl-break
LG> +++ break.c 11 Sep 2004 15:52:56 -0000 1.17.2.2
LG> @@ -53,7 +53,7 @@
LG> unsigned char check_handle_break(struct dhdr FAR **pdev)
LG> {
LG> - unsigned char c;
LG> + unsigned char c = 0;
LG> if (ctrl_break_pressed() ||
LG> (c = (unsigned char)ndread(&syscon)) == CTL_C ||
LG> *pdev != syscon && (c = (unsigned char)ndread(pdev)) == CTL_C)
Wrong. If no Ctrl-break (ctrl_break_pressed() returns false), then
called second part of condition (c = ndread()). So, my code _is_ correct,
and this extra assignemnt is useless code spending.
LG> + static struct { /* Boot sector of a RAM-Disk */
LG> + UBYTE dummy1[3]; /* HIMEM.SYS uses 3, but FDXMS uses 2 */
LG> + char Name[5];
[...]
LG> + } VDISK_BOOT_SECTOR = {
LG> + {0xcf,' ',' '},
LG> + {"VDISK"},
Unfortunately, newer standards prohibit to assign nonterminated strings
to arrays (ie. 5-character string "VDISK" _plus_ trerminating zero not fit
into 5-bytes array, so newer compilers should compain).
-------------------------------------------------------
This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170
Project Admins to receive an Apple iPod Mini FREE for your judgement on
who ports your project to Linux PPC the best. Sponsored by IBM.
Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php
_______________________________________________
Freedos-kernel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/freedos-kernel
