lib/47398: Cursor positioning in a curses form depends on the last refreshed window

[gnrp%komkon2.de@localhost]

>Number:         47398
>Category:       lib
>Synopsis:       Cursor positioning in a curses form depends on the last 
>refreshed window
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    lib-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Jan 03 21:50:00 +0000 2013
>Originator:     Julian Fagir
>Release:        NetBSD 6.0-RELEASE
>Organization:
>Environment:
>Description:
When you have a form, ncurses will set the cursor automatically to the selected 
field, no matter what window you last refreshed.

When you have a form in nbcurses, it will set the cursor only to this field 
when you refresh the subwindow it is in last.

Is this a bug or just an intended difference between nbcurses and ncurses?
>How-To-Repeat:
Compile this piece of code with -lform, run it on a ncurses and a nbcurses 
machine. See how the cursor position is always somewhere. Then, recompile it 
with the one comment commented out, and see how the cursor position is on the 
field.

#include <form.h>
#include <form.h>
#include <string.h>
#include <stdlib.h>
#include <unistd.h>
#include <stdio.h>

int
main(int arg, char *argv[])
{
    int ch, j, i = 0;
    WINDOW *mainwin, *subwin;
    FIELD *fields[2];
    FORM *form;
    char buf[512];

    initscr();
    keypad(stdscr, TRUE);
    noecho();
    cbreak();
    mainwin = newwin(20, 70, 2, 2); 
    box(mainwin, 0, 0); 
    mvwprintw(mainwin, 1, 1, "Creating window...");
    wrefresh(mainwin);

    fields[0] = new_field(1, 20, 2, 1, 0, 0); 
    fields[1] = NULL;
    form = new_form(fields);
    set_form_win(form, mainwin);
    subwin = derwin(mainwin, 15, 60, 2, 2);
    set_form_sub(form, subwin);
    set_field_back(fields[0], A_UNDERLINE);
    refresh();

    post_form(form);
    wrefresh(mainwin);

    buf[0] = '\0';

    while ((ch = getch()) != '\n') {
        switch (ch) {
            case KEY_LEFT:
                form_driver(form, REQ_LEFT_CHAR);
                i--;
                break;
            case KEY_RIGHT:
                if (buf[i] != '\0') {
                    form_driver(form, REQ_RIGHT_CHAR);
                    i++;
                }   
                break;
            case KEY_END:
                form_driver(form, REQ_END_LINE);
                i = strlen(buf);
                break;
            case KEY_HOME:
                form_driver(form, REQ_BEG_LINE);
                i = 0;
                break;
            case '\b':
            case KEY_BACKSPACE:
                form_driver(form, REQ_DEL_PREV);
                buf[i] = '\0';
                i--;
                break;
            default:
                if (i < sizeof(buf) - 2) {
                    form_driver(form, ch);
                    for (j = strlen(buf); j >= i; j--)
                        buf[j+1] = buf[j];
                    buf[i] = ch; 
                    i++;
                }   
                break;
        }   
        refresh();
        wrefresh(mainwin);
//        wrefresh(subwin);
    }   

    free_field(fields[1]);
    unpost_form(form);
    free_form(form);
    endwin();

    return 0;
}
>Fix: