i8087 / mlib Goto Github PK

I have found another error in the MikeOS documentation.

All disk functions use the carry flag to report a disk error, except os_get_file_list which returns a blank string. However, the MikeOS documentation fails to mention that os_create_file and os_remove_file use the carry flag to indicate an error.

I checked your implementation and found your 'load_file', 'create_file' and 'remove_file' functions do not handle the carry flag. Additionally, in 'get_file_size' you return zero as the file size if a disk error occurs, which may be misleading.

Would probably be desirable to put the mikedev.inc defined keywords into mlib.h for use at the C level.

The MikeOS function long_int_negate takes and returns a 32-bit value held in DX:AX Unfortunately, SmallerC doesn't support long types, so a special structure should be used and implemented like the one below.

typedef struct b_int {
    int low;
    int high;
} b_int;

argc and argv aren't handled properly in crt0.asm.

MikeOS 4.6 will require an additional parameter for the function os_input_string. This should be implement soon as it's a quick fix.

Running a program built with mlib will make MikeOS freeze.

Finish adding all of the functions for the following categories:

• Disk
• Keyboard
• Math
• Misc
• Ports
• Screen
• String
• Sound
• BASIC

Same problem as issue #3.

The function string_parse returns multiple strings. A simple solution to the problem would be to create a structure that points to all of the strings.