readline

[silderan]

readline

[silderan]

Hi.

readyRead function is not the apropiate signal to connect to. Data comming from telnet server needs to be parsed/processed because it could come with telnet specific bye codes such as screen resize information or scape bytes.

So, you need to connect to newData signal where telnet protocol bytes are stripped out.

Furthermore than that, parameters from newData need to be adapted properly for stream classes (as readLine belongs to that kind of classes, isnt' it?)
That's because QTexStream spects zero terminated char array in constructor parameter.
And, remember that you are not sure if the char * that are you passing to the QTexStream constructor is a "correct" and "parseable" text. Starting that it's not a zero-terminated string, the array could contain unfinished Unicode characters. That means, that you can receive a newData call with just 1 byte of the two bytes of one unicode character. Without going so long, new line could be double byte: \n\r or \r\n.
So, you need to "clean up" incomming data before pass it to high-level class such as QTextStream.

Yeah, QTexStream could seem to you that it is a low-level class. But remember that networking is much more low-level :) It's a "layer" below.

The procedure to parse incoming data, at ANY layer is the same:

1. Buffer it (static storage because it must persist between asyncronous calls)
2. Parse buffered data looking for a REAL error or USABLE data. If no enough data to do this, return and wait for new data.
3. Pass this data to a upper layer.
4. Discard JUST data processed
5. Go to step 2.