https://bugs.freedesktop.org/show_bug.cgi?id=72576

          Priority: medium
            Bug ID: 72576
                CC: [email protected]
          Assignee: [email protected]
           Summary: src/dbus/server/server.cpp:495: possible bad if test ?
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: [email protected]
          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:
You are on the CC list for the bug.
You are the assignee for the bug.
_______________________________________________
Syncevolution-issues mailing list
[email protected]
https://lists.syncevolution.org/mailman/listinfo/syncevolution-issues

Reply via email to