Bug in Qt4.3.3 QXmlSimpleReader::reset() ?

[Shawn]

I first post a thread here: http://www.qtcentre.org/forum/f-qt-p...sax2-7517.html

I just compiled Qt 4.3.3 open source edition with Visual C++ 2005 Express. I tried a small "hello world" example and it works fine.

But when I tried to compile my previous project using qt4.2.2 commercial with MSVS2005, I run into a problem exactly in the above thread.
[HTML]Parser error at line 1, column 1: unexpected end of file.[/HTML]

I have used QXmlSimpleReader::reset() before the second parsing, seems it doesn't work.
Then I tried QFile::seek(0), it turns out fine.

pretty strange, it is a bug in qt4.3.3 ?

[Shawn]

Any body answer this question ?

[jacek]

Could that be another reincarnation of bug #166278?

[Shawn]

Could that be another reincarnation of bug #166278?

Yes I think so

[jacek]

Could you prepare a small compilable example that reproduces the problem?

[Shawn]

Could you prepare a small compilable example that reproduces the problem?

Attatched is a small example reproduces the problem

part of it

QFile file("Resources/book.xml");
	QXmlInputSource inputSource(&file);
	QXmlSimpleReader reader;
	Parser1 handler1;
	reader.setContentHandler(&handler1);
	reader.setErrorHandler(&handler1);
	reader.parse(inputSource);
 
// QXMLSimpleReader::reset() dosen't work
	//inputSource.reset();
 
//while QFile::seek(0) works
	file.seek(0);
 
	Parser2 handler2;
	reader.setContentHandler(&handler2);
	reader.setErrorHandler(&handler2);
	reader.parse(inputSource);

QFile file("Resources/book.xml");
	QXmlInputSource inputSource(&file);
	QXmlSimpleReader reader;
	Parser1 handler1;
	reader.setContentHandler(&handler1);
	reader.setErrorHandler(&handler1);
	reader.parse(inputSource);

// QXMLSimpleReader::reset() dosen't work
	//inputSource.reset();

//while QFile::seek(0) works
	file.seek(0);

	Parser2 handler2;
	reader.setContentHandler(&handler2);
	reader.setErrorHandler(&handler2);
	reader.parse(inputSource);

To copy to clipboard, switch view to plain text mode 

[jacek]

I've just tested it and indeed it doesn't work on Qt 4.3.3. Please send that to the Trolls (qt-bugs @ ... ), so that they can fix it (I would just add a small comment that working version doesn't output anything and the broken one displays a message box).

[Shawn]

I've just tested it and indeed it doesn't work on Qt 4.3.3. Please send that to the Trolls (qt-bugs @ ... ), so that they can fix it (I would just add a small comment that working version doesn't output anything and the broken one displays a message box).

Thank you. Do you think the example is clear enough to confirm that it is a bug?

[jacek]

Do you think the example is clear enough to confirm that it is a bug?

Well... you could minimize it a bit further.

I've just made a simple test and changed main.cpp like this:

{ // <--- added {
        Parser1 handler1;
        reader.setContentHandler(&handler1);
        reader.setErrorHandler(&handler1);
        reader.parse(inputSource);
} // <--- added }

{ // <--- added {
        Parser1 handler1;
        reader.setContentHandler(&handler1);
        reader.setErrorHandler(&handler1);
        reader.parse(inputSource);
} // <--- added }

To copy to clipboard, switch view to plain text mode 

and the problem still occurs.

reset() looks like this:

void QXmlInputSource::reset()
{
    d->nextReturnedEndOfData = false;
    d->pos = 0;
}

void QXmlInputSource::reset()
{
    d->nextReturnedEndOfData = false;
    d->pos = 0;
}

To copy to clipboard, switch view to plain text mode 

so obviosly it doesn't touch the file.

[Shawn]

Sorry for late but I just found some time ...
I've report it as a bug to Trolltech.