Re: [Lost] [Patch] vterm: textterm in vterm integriert

[Kevin Wolf <mail@xxxxxxxxxxxxx>]

> +/**
> + * Ausgabe fuer ein vterminal vorbereiten
> + */
> +bool vterm_output_init(vterminal_t* vterm, size_t buffer_lines)
> +{
> +    size_t buffer_size;
> +    size_t i;
> +    videomem_cell_t c;
> +    vterm_output_t* out = &(vterm->output);
> +
> +    // Struktur initialisieren
> +    out->buffer_lines = buffer_lines;
> +    out->buffer_pos.line = 0;
> +    out->buffer_pos.column = 0;
> +    
> +    out->active = FALSE;
> +    
> +    out->screen_redraw = FALSE;
> +    out->screen_width = 80;
> +    out->screen_height = 25;

SCREEN_WIDTH_MAX und SCREEN_HEIGHT_MAX, wenn es schon konstant ist

> +/**
> + * Ausgaben eines Prozesses fuer ein Virtuelles Terminal verarbeiten
> + * 
> + * @param data Pointer auf die auszugebenden Daten
> + * @param length Laenge der auszugebenden Daten

In Bytes oder in Zeichen? ;-) Okay, ja, meinetwegen auch nur ASCII vorerst.

> +/**
> + * Eine einzelne Zeile aus dem Puffer direkt in den Videospeicher kopieren
> + *
> + * @param line Zeilennummer
> + */
> +static void screen_draw_line(vterm_output_t* out, size_t line)

Du wolltest doch dazuschreiben, worauf bezogen.

> +/**
> + * Dafuer sorgen, dass das Bildschirmfenster so angepasst wird, dass die
> + * Pufferposition pos sichtbar ist.
> + *
> + * @param pos
> + */
> +static void screen_adjust(vterm_output_t* out, position_t pos)

Ähm, ja. Willst du den Parameter pos dokumentieren oder nicht? So ist es 
jedenfalls Mist. ;-)

> +/**
> + * Einzelnes Zeichen aus einem Buffer auslesen
> + *
> + * @param buffer
> + * @param position

*hust*

Aber nachdem das mehr oder weniger alles Schönheitskorrekturen sind, 
kannst du den nächsten Versuch direkt einchecken.