| Priority | medium |
|---|---|
| Bug ID | 72576 |
| CC | syncevolution-issues@syncevolution.org |
| Assignee | syncevolution-issues@syncevolution.org |
| Summary | src/dbus/server/server.cpp:495: possible bad if test ? |
| Severity | normal |
| Classification | Unclassified |
| OS | All |
| Reporter | dcb314@hotmail.com |
| Hardware | All |
| Status | NEW |
| Version | 1.3.99.3 |
| Component | SyncEvolution |
| Product | SyncEvolution |
I just ran the static analyser over syncevolution-1.3.99.3
It said many things, including
[src/dbus/server/server.cpp:495] -> [src/dbus/server/server.cpp:495]: (style)
Same expression on both sides of '<='.
Source code is
if (session && session->getPriority() <= session->getPriority()) {
Suggest code rework. I notice there is a parameter of the routine
called session and a local variable with the same name. Odd.
I also checked the newest source code and the bug appears there too.