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.


You are receiving this mail because: