diff options
| author | Andrea Diamantini <adjam7@gmail.com> | 2012-07-06 23:06:13 +0200 |
|---|---|---|
| committer | Andrea Diamantini <adjam7@gmail.com> | 2012-07-06 23:06:13 +0200 |
| commit | 8dcaa124889dbefc80bbd21eca5e65623b22ae9f (patch) | |
| tree | 93ddf3d92fef382ed0aa0c372c20d6320893a9f5 /src | |
| parent | Fix adblock hide behavior (diff) | |
| download | rekonq-8dcaa124889dbefc80bbd21eca5e65623b22ae9f.tar.xz | |
cppcheck: suspicious code!
it seems we are executing the same code in an "if" and in the related
"else" tense. What about here?
CCMAIL: siteshwar@gmail.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/sync/operasynchandler.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/sync/operasynchandler.cpp b/src/sync/operasynchandler.cpp index 50d671f8..08c3e542 100644 --- a/src/sync/operasynchandler.cpp +++ b/src/sync/operasynchandler.cpp @@ -621,6 +621,7 @@ void OperaSyncHandler::handleLocalGroup(const KBookmarkGroup &root, const QDomEl QString id = getChildString(child, "id"); + // FIXME: What about here??? if (grandChild.isNull()) { handleLocalGroup(current.toGroup(), grandChild, id); |