Merge branch 'cmenu-video' into hdt-next

Conflicts:
com32/hdt/hdt-cli-disk.c
com32/hdt/hdt-common.c

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

Merge branch 'bootloader-detection' into hdt-next

Conflicts:
com32/hdt/hdt-cli-disk.c
com32/hdt/hdt-common.h
com32/hdt/hdt-menu-disk.c

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: use ASCII characters for scrollbars

Use ^ and v as hints for scrolling. Use \141 from the Special Characters and
Line Drawing Character Set as the scrollbox.

Note: printf "\016\141\017" doesn't seem to work well on xterm, although
it does work on the vga console.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: do not invoke help if helpid is 0xFFFF (complex.c)

0xFFFF is invalid (no help)

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: do not invoke help if helpid is 0xFFFF

0xFFFF is invalid (no help).

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: redraw the current menu after exiting the keys_handler

After exiting the keys_handler, reset_ui() is not enough since it
only resets the status line and title. The current menu needs to be redrawn.

Note: only a submenu will be redrawn, not its parents. This means that not
all the UI will be displayed after exiting the handler if invoked from whithin a
submenu. We have already this problem nowadays when exiting a submenu
of a submenu. The root parent is not redrawn, leading to some ugly states
(the background color overrides part of the root menu).
This will be fixed when we implement a real hierarchy (each submenu having
access to its parents).

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: remove extra LF

In a few places, extra LFs cause the menu to scroll ('\n' in the status bar).
Remove them (they are not needed anyway).

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: optimize redraw to be faster

cmenu currently redraws everything after each key press. This doesn't scale
well, especially over serial. Instead, redraw only the previous/current selected
entries. A full redraw is still needed when scrolling though.

Misc.: add CTRL-L combo to force a redraw of the current menu.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: force refresh after checkbox_handler in complex.c

complex.c has a special checkbox_handler that disables mutual exclusive
entries. This works today because each key press triggers a redraw of the whole
menu. We can't rely on this if we want to optimize the redraw.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: redraw menu when exiting keys_handler

The keys_handler can change the UI (e.g. the help system draws a black
background by default). We need to redraw the background and title on exit.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: pad help filename with 0 instead of spaces

The manual states that the help filename format is hlp<NNNNN>.txt where
<NNNNN> is the helpid.

If the helpid contains less than 5 digits, pad with zero, not spaces,
i.e. look for hlp00023.txt, not hlp   23.txt.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: remove obsolete com32io code

Remove legacy scrollupwindow definition and checkkbdbuf declaration.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: make the help menu actually work

With the recent changes in cmenu, the help feature was completely broken.
Fix it.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: fix build due to cmenu keyboard changes

Sames changes as the ones needed on the cmenu examples.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: add function to scroll up multiple lines

Impact: API change. scrollup() has been renamed scrollup_once().

Use scrollup(nb_of_lines) to specify the number of lines the window needs
to scroll.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: fix handlers signature in the examples

The keys handler accepts three arguments: the current menu system, the current
menu entry and the key pressed.

Misc: fix timeout in complex.c to actually be 10 seconds.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: use get_key library for keyboard functions

Syslinux already uses a get_key library to handle stdin. Let's leverage it.
cmenu now works over serial completely.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: correctly reset attributes

When resetting the attributes, we were updating last_attr to the unknown value
0x300, which doesn't always work.

For instance, when requesting reset and normal attributes, i.e.
"\e[0;30;47m" with the previous attribute being "\e[1;30;47m", we where printing
"\e[0;47m", which behaves like "\e[0;39;47m". This is incorrect.

This patch adds a flag to explicitly set background and foreground colors after
a reset.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: reset_colors should use csprint

cprint_vga2ansi takes only a char, not a string.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: fill backgorund with printable character

To have a blue background on the console (serial), we need to fill
the screen with a printable character (e.g. space).

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: remove legacy CHABSATTR/CHRELATTR code

CHABSATTR and CHRELATTR have been replaced by SO/SI.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: remove legacy box drawing code

Lines are now printed via the Line Drawing Character Set.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: unify normal menu/radio menu handling

A lot of code was duplicated between regular and normal menus
(menu printing and selection handling). This patch unifies this logic
by adding an extra bool in printmenu() and getmenuoption() to distinguish
between the two types of menus.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: fix Login/Password and Kernel Arguments lines (complex.c)

Update the cmenu example complex.c to use clear_line(): cprint behavior has
changed w.r.t. cursor positioning (it is automatically advanced now).

Note: need to reset the colors before clearing the line since the status bar is
printed just before the Kernel Arguments line and it changes the background
color.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: add reset_colors function

Purpose of reset_colors is to reset the attribute to VGA 0x07.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: turn off auto wrap

Auto wrap made the menu scroll because of the status line (when printing
the last character of the bottom right of the screen, the ansicon scrolled
the whole menu).

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

ansi: add support for no-wrap mode, cleanups

Add support for nowrap mode CSI ? 7 l (unfortunately, on ANSI.SYS it
is CSI = 7 l; which means that the only way to support both is to
print both sequences and then erase the resulting garbage.)

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge commit 'origin/libansi' into cmenu-video

Merge branch 'cmenu-video' of git://git.zytor.com/users/pam/hdt-pierre into cmenu-video

cmenu: use printf instead of csprint with default attribute

We don't change the attribute in the help screen. Refactor the code
to simply use printf.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: use putchar instead of putch when using default attribute

There is no real need to do putch('x', 0x07), 0x07 being the default
VGA attribute. Use putchar instead.

Note that showhelp will trigger a drawbox first with 0x07 as attribute.
Since we don't update the attribute, it is guaranteed to still be 0x07
when invoking putch.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: remove call to legacy getnumcols (menu)

Use getscreensize instead.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: use getscreensize to access number of rows/columns

The simple menu system already uses getscreensize. Another step towards
unification.

Impact: exported getnumcols and getnumrows are now deprecated.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: add const qualifier in cpu_dev (cpuid)

The Syslinux build turns on -Wwrite-strings, which gives string constants
the type const char[length].

Note: -Wall doesn't trigger this warning.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: add missing header in ata.c

This fixes an implicit definition warning.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: fix signedness warnings

host_bus_type and interface_type are uint8_t. Force a cast when calling remove_spaces
on these fields.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: unify disk headers in hdt-common.h

Both cli and menu need to access disk functions. Include disk headers
in hdt-common.h, instead of duplicating code.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: remove get_error declaration from disk/util.h

In d1890409251949b80c2026557b9b12ca00d55144, it seems we
have forgotten to update util.h.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: fix cpuid.h Once-Only Header wrapper

It seems that the cpuid.h had a misplaced #endif.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: share PAGE_SIZE definition with cpuid.h

swsusp detection needs PAGE_SIZE. cpuid.h is actually already defining it.
Since both swsusp and cpuid are taken from the Linux kernel, it is a good idea to
share it (they really need to be equal).

We are going to rely on cpuid.h. Creating a page.h for this single definition seems
overkilling and the MMU is part of the CPU on x86 anyways.

Impact: fix compiler warning (redefinition).

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: fix hdt-cli.c compilation warning

We are missing a const qualifier.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: fix compiler warnings

Fix "warning: function declaration isn't a prototype" warnings.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: switch to libansi

Make hdt use libansi.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: switch to libansi

Make cmenu use libansi.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: add missing prototype in the header

clearwindow() was not presented in libansi.h.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: build libansi.o

Hook libansi.o into the lib/ build.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

libansi: move to lib directory, as it is MIT, not GPL licensed

gplinclude/gpllib should be used for external, GPL, libraries
only. libansi has been developed for Syslinux and is MIT licensed.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: add more ANSI functions

Import the functions that I introduced in cmenu.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: add a new ansi library

HDT and cmenu are currently duplicating code: they both define their
own mini ansi library.

This patch extracts the HDT ansi lib into gpllib as a first step towards
unification.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: reset attributes instead of using ANSI sequence 22

hpa advised not to use ANSI sequence 22, as it may not work everywhere.
Instead, let's reset attributes and set last_attr to unknown. The next
background/foreground tests are then guaranteed to fire. We take a small
hit in terms of performance though.

Misc.: change initialization of last_attr to 0x300 (unknown) to trigger
properly the tests the first time.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: fix background color

The menu highlights some characters to mark them as key shortcuts.
To switch the bold attribute off, we were printing the reset code,
resetting the background color in the same time.

Use ANSI sequence 22 instead.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

Revert "ansicon: allow to pass page # to write functions"

We shouldn't be using any pages, as there is no support on the
serial line.

This reverts commit e9b317fd0dc87dc93a49bc40520f4bdd0264d746.

cmenu: use 80x25 as terminal size by default

Let's not rely on the geometry returned by the BIOS for the
terminal size - we don't know anything about the one used via serial.

We now default to 80x25, regardless of what the BIOS returns.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: misc. cleanups

Fix a compiler warning and a header documentation.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: remove multi pages support

The cmenu used to display the help on a separate page. Since we
can't have multi pages on the serial line, get rid of them completely.

This has a fairly big impact as the external cmenu API has changed.
adv_menu.tpl and HDT have been patched accordingly.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: use < for menu exit character

Use plain '<' as the "back" symbol.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: fix menu separators

Use the alternate character set to draw horizontal lines (menu
separator). Refactor the c*print functions in the same time as they
were severely broken.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: use > instead of » for submenus

» doesn't exist in the alternate character set. Use plain
> instead.

Misc.: fix SO/SI invocation, putchar takes a char, fputs a string.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: remove extra getboxchars call

getboxchars is not used anymore in drawbox. Let's get rid of it.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: build cleanups

Fix build warnings and remove old cls declaration.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: refactor cswprint function

Now that cprint is smarter, cswprint can be simplified.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: refactor printmenuitem function

Simplify the printmenuitem function, now that putch is smarter.

cmenu: refactor beep function

Use \007 to make the system beep. Note "\007" works, but not '\007'.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: refactor cprint functions

Some refactoring:

* Make cprint_vga2ansi static, as it is an internal function.
* Simplify the logic in cprint().

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: use VT-100 alternate character set for drawing boxes

Use the Special Characters and Line Drawing Character Set as
described in http://www.vt100.net/docs/vt102-ug/table5-13.html
to draw boxes around the menus.

Note that to work in xterm, G1 needs to be initialized (to point
to the alternate character set). This is done in the cls function
("\033)0").

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: remove unused bit_reverse function

Not needed anymore: we use a lookup table (ansi_char)
in cprint_vga2ansi now to reverse Background and Foreground bits..

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: better implementation of vga->ansi

hpa suggested a better implementation, that also fixes some color
issues and invalid ANSI codes. Thanks hpa!

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: implement cprint using printf

First step towards a usable menu. Color is now sent over serial.

Testing Done: launched HDT over serial. The background got drawn blue,
as expected.

Note: vga attributes need to be converted to ansi codes.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: remove unused getchar function

char getchar(void) doesn't seem to be used.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: implement scrollup using SU escape sequence

Use the ANSI SU - Scroll Up escape sequence to scrollup the window.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

cmenu: implement gotoxy using escape sequences

Use ansicon and the ANSI CUP - CUrsor Position escape sequence
to implement the gotoxy function.

Note: page switching is not supported (yet).

Testing Done: ran com32/cmenu/test.c32 in qemu.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

ansicon: allow to pass page # to write functions

Some video modes support up to 8 different pages, the current one being
stored at 0x462.

The display page value is passed in BH when invoking the INT 10h Video
Service routines. This patch changes the interface of ansicon_write_char
and ansicon_set_cursor to allow the caller to specify the page.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: add bootloader detection (menu)

Print the bootloader ID in the partition submenu if we recognize it.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: add bootloader detection (CLI)

When iterating through all partitions (e.g. `show disks'),
print the bootloader ID if we recognize it.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: add bootloader detection

Given an (active) partition, look for a bootloader ID.

Syslinux prints a string (SYSLINUX or EXTLINUX) between bytes
3 and 10.

After some random tests, it seems that Windows™98/98SE and Windows™ME
put MSWIN4.1 at the same place. Internet seems to confirm it:

   http://www.geocities.com/thestarman3/asm/mbr/MSWin41BRinHexEd.htm

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: rename __BOOTLOADERS_H_ to __MBRS_H_ in mbrs.h

We had forgotten to update the #define when renaming the file.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: rename bootloader detection to MBR detection

bootloaders.c really inspects the MBR, not the bootloader itself.
Let's not confuse the two of them.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: use four bytes to discover the MBR

Syslinux and Master Boot LoaDeR share the first 2 bytes. Use one more
to distinguish between the two of them.

To keep standards types, uint16_t is being replaced by uint32_t.
The extra byte is not used yet.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

NEWS: update for 3.83

hdt: enhance error reporting in the CLI (disk)

parse_partition_table can fail for two reasons: disk I/O
error or unrecognized partition layout (non msdos, corrupt, ...).

In the latter case, we were displaying

   Error parsing disk 0x80
   parse_partition_table: error 0

which is confusing. We now display

   Disk 0x80: unrecognized partition layout

or the errno_disk in case of I/O errors.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: make get_error behaves like perror

This simplifies buffer handling.

Misc.: clean some old error code handling.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: fix CLI crash when parsing unsupported geometries

Remove extra free() calls: these were needed before the introduction
of the disklib errno. We apparently forgot to remove them.

Reported-by: Gert Hulselmans <hulselmansgert@gmail.com>
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: right-align information about each disk (CLI)

Align C/H/S, EDD and Bootloader fields.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: enhance user experience (CLI) - try harder

Alright - let's try again.

  * Manually increment the display_line_nb counter in the disk CLI
    (we are doing a multi line more_printf!)
  * Add the pause_printf() macro to pause the CLI as needed

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: fix comment

buffer_ptr has been renamed to s.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: fix comment

buffer_ptr has been renamed to s.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: enhance user experience (CLI)

Some users have reported that reading the CLI can be difficult,
especially when spitting out a lot of information.

This patch:

  * adds a missing reset_more_printf() in the disk CLI
  * reduces the total scrolling to 20 lines (seems better in my testing)
  * removes a reset_more_printf() in the show disk function. This can potentially
    be bad when doing a lot of show disk 0xXX (especially after 4 in a row)
    but enhances the display when running show disks with lots of disks.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: enhance user experience (CLI)

Some users have reported that reading the CLI can be difficult,
especially when spitting out a lot of information.

This patch:

  * adds a missing reset_more_printf() in the disk CLI
  * reduces the total scrolling to 20 lines (seems better in my testing)
  * removes a reset_more_printf() in the show disk function. This can potentially
    be bad when doing a lot of show disk 0xXX (especially after 4 in a row)
    but enhances the display when running show disks with lots of disks.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: tweak bootloader display (menu)

The extra "id" makes the menu too large under Qemu, removing it
enhances the user experience.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: fix bootloader entry alignment

We're missing a space here.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: print bootloader name (menu)

Add bootloader information in the disk menu.

Misc.: fix compilation warning by refactoring the compute_disks interface
(remove unneeded hardware->disk_info argument).

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: fix compilation warning (disk/bootloaders.h)

struct driveinfo is defined in disk/geom.h.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: print bootloader name (CLI)

`show disks' in the CLI now prints bootloader information, e.g.

DISK 0x83:
  C/H/S: 1023 cylinders, 255 heads, 63 sectors/track
  EDD:   Version: 30
         Size:  10 GiB, 512 bytes/sector, 63 sectors/track
         Host bus: ISA, Interface type: ATA
  Bootloader: Grub (id 0xEB48)

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: bootloader id should be uint16_t

The id is of the form 0xABCD: use uint16_t instead of int.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: detect bootloader ids when probing disks

Add a bootloader_ids[] array in the hardware structure to store bootloader
ids when probing the disks.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

gpllib: add bootloader detection

The first two bytes of the hard drive can help identifying the boot code
installed in the MBR.

Values from the boot_info_script:

    http://sourceforge.net/projects/bootinfoscript/

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: enhance error reporting in the CLI (disk)

parse_partition_table can fail for two reasons: disk I/O
error or unrecognized partition layout (non msdos, corrupt, ...).

In the latter case, we were displaying

   Error parsing disk 0x80
   parse_partition_table: error 0

which is confusing. We now display

   Disk 0x80: unrecognized partition layout

or the errno_disk in case of I/O errors.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: make get_error behaves like perror

This simplifies buffer handling.

Misc.: clean some old error code handling.

Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: fix CLI crash when parsing unsupported geometries

Remove extra free() calls: these were needed before the introduction
of the disklib errno. We apparently forgot to remove them.

Reported-by: Gert Hulselmans <hulselmansgert@gmail.com>
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>

hdt: 0.3.4

Impact: New release

0.3.4 is now out !