Skip to content

Commit 34c440b

Browse files
committed
ticket:4291 Don't use string buffer for xml data.
1 parent 69763e4 commit 34c440b

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

OMEdit/OMEditGUI/Simulation/SimulationOutputHandler.cpp

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,6 @@ SimulationOutputHandler::SimulationOutputHandler(SimulationOutputWidget *pSimula
299299
mXmlSimpleReader.setErrorHandler(this);
300300
mpXmlInputSource = new QXmlInputSource;
301301
mpXmlInputSource->setData(simulationOutput.prepend("<root>"));
302-
mOutputBuffer = mpXmlInputSource->data();
303302
mXmlSimpleReader.parse(mpXmlInputSource, true);
304303
}
305304

@@ -314,7 +313,6 @@ SimulationOutputHandler::~SimulationOutputHandler()
314313
void SimulationOutputHandler::parseSimulationOutput(QString output)
315314
{
316315
mpXmlInputSource->setData(output);
317-
mOutputBuffer.append(mpXmlInputSource->data());
318316
mXmlSimpleReader.parseContinue();
319317
}
320318

@@ -383,11 +381,10 @@ bool SimulationOutputHandler::endElement(const QString &namespaceURI, const QStr
383381
bool SimulationOutputHandler::fatalError(const QXmlParseException &exception)
384382
{
385383
// read the error message
386-
QString error = QString("Fatal error on line %1, column %2: %3\nXML ::\n%4")
384+
QString error = QString("Fatal error on line %1, column %2: %3")
387385
.arg(exception.lineNumber())
388386
.arg(exception.columnNumber())
389-
.arg(exception.message())
390-
.arg(mOutputBuffer);
387+
.arg(exception.message());
391388
// construct the SimulationMessage object with error
392389
SimulationMessage *pSimulationMessage;
393390
if (mpSimulationOutputWidget->isOutputStructured()) {

OMEdit/OMEditGUI/Simulation/SimulationOutputHandler.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,6 @@ class SimulationOutputHandler : private QXmlDefaultHandler
100100
int mLevel;
101101
SimulationMessage* mpSimulationMessage;
102102
QMap<int, SimulationMessage*> mSimulationMessagesLevelMap;
103-
QString mOutputBuffer;
104103
SimulationMessageModel *mpSimulationMessageModel;
105104
QXmlSimpleReader mXmlSimpleReader;
106105
QXmlInputSource *mpXmlInputSource;

0 commit comments

Comments
 (0)