X-Git-Url: https://git.cworth.org/git?a=blobdiff_plain;f=gui%2Fapitrace.cpp;h=671fc683887fa35fbbbda15bde9b92cae1df5b43;hb=caa8428eb4c4292d3ea53405d4dc9c3c3f21f10a;hp=bff1686a9a3076add8058fcad199a483b9b5e873;hpb=8f98c3a529e7ef88e4111ef22cf8411916a9a065;p=apitrace diff --git a/gui/apitrace.cpp b/gui/apitrace.cpp index bff1686..671fc68 100644 --- a/gui/apitrace.cpp +++ b/gui/apitrace.cpp @@ -8,8 +8,7 @@ #include ApiTrace::ApiTrace() - : m_frameMarker(ApiTrace::FrameMarker_SwapBuffers), - m_needsSaving(false) + : m_needsSaving(false) { m_loader = new TraceLoader(); @@ -23,16 +22,28 @@ ApiTrace::ApiTrace() SIGNAL(frameContentsLoaded(ApiTraceFrame*,QVector,quint64)), this, SLOT(loaderFrameLoaded(ApiTraceFrame*,QVector,quint64))); + connect(m_loader, SIGNAL(guessedApi(int)), + this, SLOT(guessedApi(int))); connect(m_loader, SIGNAL(finishedParsing()), this, SLOT(finishedParsing())); - connect(this, SIGNAL(loaderSearchNext(int,QString,Qt::CaseSensitivity)), - m_loader, SLOT(searchNext(int,QString,Qt::CaseSensitivity))); - connect(this, SIGNAL(loaderSearchPrev(int,QString,Qt::CaseSensitivity)), - m_loader, SLOT(searchPrev(int,QString,Qt::CaseSensitivity))); + connect(this, SIGNAL(loaderSearch(ApiTrace::SearchRequest)), + m_loader, SLOT(search(ApiTrace::SearchRequest))); connect(m_loader, - SIGNAL(searchResult(ApiTrace::SearchResult,ApiTraceCall*)), + SIGNAL(searchResult(ApiTrace::SearchRequest,ApiTrace::SearchResult,ApiTraceCall*)), this, - SLOT(loaderSearchResult(ApiTrace::SearchResult,ApiTraceCall*))); + SLOT(loaderSearchResult(ApiTrace::SearchRequest,ApiTrace::SearchResult,ApiTraceCall*))); + connect(this, SIGNAL(loaderFindFrameStart(ApiTraceFrame*)), + m_loader, SLOT(findFrameStart(ApiTraceFrame*))); + connect(this, SIGNAL(loaderFindFrameEnd(ApiTraceFrame*)), + m_loader, SLOT(findFrameEnd(ApiTraceFrame*))); + connect(m_loader, SIGNAL(foundFrameStart(ApiTraceFrame*)), + this, SIGNAL(foundFrameStart(ApiTraceFrame*))); + connect(m_loader, SIGNAL(foundFrameEnd(ApiTraceFrame*)), + this, SIGNAL(foundFrameEnd(ApiTraceFrame*))); + connect(this, SIGNAL(loaderFindCallIndex(int)), + m_loader, SLOT(findCallIndex(int))); + connect(m_loader, SIGNAL(foundCallIndex(ApiTraceCall*)), + this, SIGNAL(foundCallIndex(ApiTraceCall*))); connect(m_loader, SIGNAL(startedParsing()), @@ -58,65 +69,26 @@ ApiTrace::~ApiTrace() { m_loaderThread->quit(); m_loaderThread->deleteLater(); - qDeleteAll(m_calls); qDeleteAll(m_frames); delete m_loader; delete m_saver; } -bool ApiTrace::isCallAFrameMarker(const ApiTraceCall *call, - ApiTrace::FrameMarker marker) -{ - if (!call) - return false; - - switch (marker) { - case FrameMarker_SwapBuffers: - return call->name().contains(QLatin1String("SwapBuffers")) || - call->name() == QLatin1String("CGLFlushDrawable") || - call->name() == QLatin1String("glFrameTerminatorGREMEDY"); - case FrameMarker_Flush: - return call->name() == QLatin1String("glFlush"); - case FrameMarker_Finish: - return call->name() == QLatin1String("glFinish"); - case FrameMarker_Clear: - return call->name() == QLatin1String("glClear"); - } - - Q_ASSERT(!"unknown frame marker"); - - return false; -} - bool ApiTrace::isEmpty() const { - return m_calls.isEmpty(); + return m_frames.isEmpty(); } QString ApiTrace::fileName() const { - if (edited()) + if (edited()) { return m_tempFileName; + } return m_fileName; } -ApiTrace::FrameMarker ApiTrace::frameMarker() const -{ - return m_frameMarker; -} - -QVector ApiTrace::calls() const -{ - return m_calls; -} - -int ApiTrace::numCalls() const -{ - return m_calls.count(); -} - -QList ApiTrace::frames() const +const QList & ApiTrace::frames() const { return m_frames; } @@ -134,43 +106,32 @@ int ApiTrace::numFrames() const int ApiTrace::numCallsInFrame(int idx) const { const ApiTraceFrame *frame = frameAt(idx); - if (frame) + if (frame) { return frame->numChildren(); - else + } else { return 0; + } } void ApiTrace::setFileName(const QString &name) { if (m_fileName != name) { m_fileName = name; + m_tempFileName = QString(); m_frames.clear(); - m_calls.clear(); m_errors.clear(); m_editedCalls.clear(); + m_queuedErrors.clear(); m_needsSaving = false; emit invalidated(); -// m_loader->loadTrace(m_fileName); emit loadTrace(m_fileName); } } -void ApiTrace::setFrameMarker(FrameMarker marker) -{ - if (m_frameMarker != marker) { - emit framesInvalidated(); - - qDeleteAll(m_frames); - m_frames.clear(); - detectFrames(); - } -} - void ApiTrace::addFrames(const QList &frames) { - QVector calls; int currentFrames = m_frames.count(); int numNewFrames = frames.count(); @@ -178,58 +139,20 @@ void ApiTrace::addFrames(const QList &frames) m_frames += frames; - int currentCalls = m_calls.count(); - int numNewCalls = 0; foreach(ApiTraceFrame *frame, frames) { frame->setParentTrace(this); - numNewCalls += frame->numChildren(); - calls += frame->calls(); } - m_calls.reserve(m_calls.count() + calls.count() + 1); - m_calls += calls; - - emit endAddingFrames(); - emit callsAdded(currentCalls, numNewCalls); -} -void ApiTrace::detectFrames() -{ - if (m_calls.isEmpty()) - return; - - emit beginAddingFrames(0, m_frames.count()); - - ApiTraceFrame *currentFrame = 0; - foreach(ApiTraceCall *apiCall, m_calls) { - if (!currentFrame) { - currentFrame = new ApiTraceFrame(this); - currentFrame->number = m_frames.count(); - currentFrame->setLoaded(true); - } - apiCall->setParentFrame(currentFrame); - currentFrame->addCall(apiCall); - if (ApiTrace::isCallAFrameMarker(apiCall, - m_frameMarker)) { - m_frames.append(currentFrame); - currentFrame = 0; - } - } - //last frames won't have markers - // it's just a bunch of Delete calls for every object - // after the last SwapBuffers - if (currentFrame) { - m_frames.append(currentFrame); - currentFrame = 0; - } emit endAddingFrames(); } ApiTraceCall * ApiTrace::callWithIndex(int idx) const { - for (int i = 0; i < m_calls.count(); ++i) { - ApiTraceCall *call = m_calls[i]; - if (call->index() == idx) + for (int i = 0; i < m_frames.count(); ++i) { + ApiTraceCall *call = m_frames[i]->callWithIndex(idx); + if (call) { return call; + } } return NULL; } @@ -237,10 +160,16 @@ ApiTraceCall * ApiTrace::callWithIndex(int idx) const ApiTraceState ApiTrace::defaultState() const { ApiTraceFrame *frame = frameAt(0); - if (!frame || !frame->hasState()) + if (!frame || !frame->isLoaded() || frame->isEmpty()) { return ApiTraceState(); + } - return *frame->state(); + ApiTraceCall *firstCall = frame->calls().first(); + if (!firstCall->hasState()) { + return ApiTraceState(); + } + + return *firstCall->state(); } void ApiTrace::callEdited(ApiTraceCall *call) @@ -284,7 +213,9 @@ void ApiTrace::save() QDir dir; emit startedSaving(); dir.mkpath(fi.absolutePath()); - m_saver->saveFile(m_tempFileName, m_calls); + m_saver->saveFile(m_tempFileName, + m_fileName, + m_editedCalls); } void ApiTrace::slotSaved() @@ -297,34 +228,37 @@ bool ApiTrace::isSaving() const return m_saver->isRunning(); } -void ApiTrace::callError(ApiTraceCall *call) +bool ApiTrace::hasErrors() const { - Q_ASSERT(call); - - if (call->hasError()) - m_errors.insert(call); - else - m_errors.remove(call); + return !m_errors.isEmpty() || !m_queuedErrors.isEmpty(); +} - emit changed(call); +void ApiTrace::loadFrame(ApiTraceFrame *frame) +{ + if (!isFrameLoading(frame)) { + Q_ASSERT(!frame->isLoaded()); + m_loadingFrames.insert(frame); + emit requestFrame(frame); + } } -bool ApiTrace::hasErrors() const +void ApiTrace::guessedApi(int api) { - return !m_errors.isEmpty(); + m_api = static_cast(api); } -void ApiTrace::loadFrame(ApiTraceFrame *frame) +trace::API ApiTrace::api() const { - Q_ASSERT(!frame->loaded()); - emit requestFrame(frame); + return m_api; } void ApiTrace::finishedParsing() { - ApiTraceFrame *firstFrame = m_frames[0]; - if (firstFrame && !firstFrame->loaded()) { - loadFrame(firstFrame); + if (!m_frames.isEmpty()) { + ApiTraceFrame *firstFrame = m_frames[0]; + if (firstFrame && !firstFrame->isLoaded()) { + loadFrame(firstFrame); + } } } @@ -332,10 +266,39 @@ void ApiTrace::loaderFrameLoaded(ApiTraceFrame *frame, const QVector &calls, quint64 binaryDataSize) { - Q_ASSERT(frame->numChildrenToLoad() == calls.size()); - emit beginLoadingFrame(frame, calls.size()); - frame->setCalls(calls, binaryDataSize); - emit endLoadingFrame(frame); + Q_ASSERT(frame->numChildrenToLoad() >= calls.size()); + + if (!frame->isLoaded()) { + emit beginLoadingFrame(frame, calls.size()); + frame->setCalls(calls, binaryDataSize); + emit endLoadingFrame(frame); + m_loadingFrames.remove(frame); + } + + if (!m_queuedErrors.isEmpty()) { + QList< QPair >::iterator itr; + for (itr = m_queuedErrors.begin(); itr != m_queuedErrors.end(); + ++itr) { + const ApiTraceError &error = (*itr).second; + if ((*itr).first == frame) { + ApiTraceCall *call = frame->callWithIndex(error.callIndex); + + if (!call) { + continue; + } + + call->setError(error.message); + m_queuedErrors.erase(itr); + + if (call->hasError()) { + m_errors.insert(call); + } else { + m_errors.remove(call); + } + emit changed(call); + } + } + } } void ApiTrace::findNext(ApiTraceFrame *frame, @@ -345,11 +308,13 @@ void ApiTrace::findNext(ApiTraceFrame *frame, { ApiTraceCall *foundCall = 0; int frameIdx = m_frames.indexOf(frame); + SearchRequest request(SearchRequest::Next, + frame, from, str, sensitivity); - if (frame->loaded()) { + if (frame->isLoaded()) { foundCall = frame->findNextCall(from, str, sensitivity); if (foundCall) { - emit findResult(SearchFound, foundCall); + emit findResult(request, SearchResult_Found, foundCall); return; } @@ -358,19 +323,23 @@ void ApiTrace::findNext(ApiTraceFrame *frame, frameIdx += 1; } + //for the rest of the frames we search from beginning + request.from = 0; for (int i = frameIdx; i < m_frames.count(); ++i) { ApiTraceFrame *frame = m_frames[i]; - if (!frame->loaded()) { - emit loaderSearchNext(i, str, sensitivity); + request.frame = frame; + if (!frame->isLoaded()) { + emit loaderSearch(request); return; } else { ApiTraceCall *call = frame->findNextCall(0, str, sensitivity); if (call) { - emit findResult(SearchFound, call); + emit findResult(request, SearchResult_Found, call); + return; } } } - emit findResult(SearchWrapped, 0); + emit findResult(request, SearchResult_Wrapped, 0); } void ApiTrace::findPrev(ApiTraceFrame *frame, @@ -380,11 +349,13 @@ void ApiTrace::findPrev(ApiTraceFrame *frame, { ApiTraceCall *foundCall = 0; int frameIdx = m_frames.indexOf(frame); + SearchRequest request(SearchRequest::Prev, + frame, from, str, sensitivity); - if (frame->loaded()) { + if (frame->isLoaded()) { foundCall = frame->findPrevCall(from, str, sensitivity); if (foundCall) { - emit findResult(SearchFound, foundCall); + emit findResult(request, SearchResult_Found, foundCall); return; } @@ -393,27 +364,144 @@ void ApiTrace::findPrev(ApiTraceFrame *frame, frameIdx -= 1; } - for (int i = frameIdx; i <= 0; --i) { + request.from = 0; + for (int i = frameIdx; i >= 0; --i) { ApiTraceFrame *frame = m_frames[i]; - if (!frame->loaded()) { - emit loaderSearchPrev(i, str, sensitivity); + request.frame = frame; + if (!frame->isLoaded()) { + emit loaderSearch(request); return; } else { ApiTraceCall *call = frame->findPrevCall(0, str, sensitivity); if (call) { - emit findResult(SearchFound, call); + emit findResult(request, SearchResult_Found, call); + return; } } } - emit findResult(SearchWrapped, 0); + emit findResult(request, SearchResult_Wrapped, 0); } -void ApiTrace::loaderSearchResult(ApiTrace::SearchResult result, +void ApiTrace::loaderSearchResult(const ApiTrace::SearchRequest &request, + ApiTrace::SearchResult result, ApiTraceCall *call) { //qDebug()<<"Search result = "<