[Bug 284907] New: KTelepathy operator< code is nonsense

David Edmundson kde at davidedmundson.co.uk
Tue Oct 25 00:44:07 UTC 2011


https://bugs.kde.org/show_bug.cgi?id=284907

           Summary: KTelepathy operator< code is nonsense
           Product: telepathy
           Version: unspecified
          Platform: Unlisted Binaries
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: NOR
         Component: common-internals
        AssignedTo: kde-telepathy at kde.org
        ReportedBy: kde at davidedmundson.co.uk


it constructs a lookup table each time, and this is incorrectly called
m_presenceSorting despite not being a member variable. Result of bad
copy+pasting.

---

Needs moving to static variable, with a static initialiser. (and probably a
static bool to say it's been initialised).

OR

get rid of the hash and replace it with a private method that returns an int
based on a switch statement.

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the KDE-Telepathy mailing list