Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c2176634 authored by Mathias Agopian's avatar Mathias Agopian Committed by Sriram Raman
Browse files

fix [3408713] Dialog window invisible sometimes 

weak pointer comparison operators were implemented wrong,
they were using the internal "unsafe" pointer. We could end up
with two "equal" weak pointer pointing to different objects.

this caused KeyedVector keyed by weak pointer to work incorrectly, in
turn causing a window that just got added to a list to be immediately
removed.

Change-Id: Ib191010c39aafa9229109e4211f6c3b2b2f9696d
parent 711404d1

include/utils/RefBase.h

+48 −14
Original line number Diff line number Diff line
@@ -31,13 +31,10 @@ template<typename T> class wp; 


// ---------------------------------------------------------------------------



#define COMPARE(_op_)                                           \

#define COMPARE_WEAK(_op_)                                      \

inline bool operator _op_ (const sp<T>& o) const {              \

    return m_ptr _op_ o.m_ptr;                                  \

}                                                               \

inline bool operator _op_ (const wp<T>& o) const {              \

    return m_ptr _op_ o.m_ptr;                                  \

}                                                               \

inline bool operator _op_ (const T* o) const {                  \

    return m_ptr _op_ o;                                        \

}                                                               \
@@ -46,12 +43,18 @@ inline bool operator _op_ (const sp<U>& o) const { \ 
    return m_ptr _op_ o.m_ptr;                                  \

}                                                               \

template<typename U>                                            \

inline bool operator _op_ (const wp<U>& o) const {              \

inline bool operator _op_ (const U* o) const {                  \

    return m_ptr _op_ o;                                        \

}



#define COMPARE(_op_)                                           \

COMPARE_WEAK(_op_)                                              \

inline bool operator _op_ (const wp<T>& o) const {              \

    return m_ptr _op_ o.m_ptr;                                  \

}                                                               \

template<typename U>                                            \

inline bool operator _op_ (const U* o) const {                  \

    return m_ptr _op_ o;                                        \

inline bool operator _op_ (const wp<U>& o) const {              \

    return m_ptr _op_ o.m_ptr;                                  \

}



// ---------------------------------------------------------------------------
@@ -275,12 +278,42 @@ public: 


    // Operators



    COMPARE(==)

    COMPARE(!=)

    COMPARE(>)

    COMPARE(<)

    COMPARE(<=)

    COMPARE(>=)

    COMPARE_WEAK(==)

    COMPARE_WEAK(!=)

    COMPARE_WEAK(>)

    COMPARE_WEAK(<)

    COMPARE_WEAK(<=)

    COMPARE_WEAK(>=)



    inline bool operator == (const wp<T>& o) const {

        return (m_ptr == o.m_ptr) && (m_refs == o.m_refs);

    }

    template<typename U>

    inline bool operator == (const wp<U>& o) const {

        return m_ptr == o.m_ptr;

    }



    inline bool operator > (const wp<T>& o) const {

        return (m_ptr == o.m_ptr) ? (m_refs > o.m_refs) : (m_ptr > o.m_ptr);

    }

    template<typename U>

    inline bool operator > (const wp<U>& o) const {

        return (m_ptr == o.m_ptr) ? (m_refs > o.m_refs) : (m_ptr > o.m_ptr);

    }



    inline bool operator < (const wp<T>& o) const {

        return (m_ptr == o.m_ptr) ? (m_refs < o.m_refs) : (m_ptr < o.m_ptr);

    }

    template<typename U>

    inline bool operator < (const wp<U>& o) const {

        return (m_ptr == o.m_ptr) ? (m_refs < o.m_refs) : (m_ptr < o.m_ptr);

    }

                         inline bool operator != (const wp<T>& o) const { return m_refs != o.m_refs; }

    template<typename U> inline bool operator != (const wp<U>& o) const { return !operator == (o); }

                         inline bool operator <= (const wp<T>& o) const { return !operator > (o); }

    template<typename U> inline bool operator <= (const wp<U>& o) const { return !operator > (o); }

                         inline bool operator >= (const wp<T>& o) const { return !operator < (o); }

    template<typename U> inline bool operator >= (const wp<U>& o) const { return !operator < (o); }



private:

    template<typename Y> friend class sp;
@@ -294,6 +327,7 @@ template <typename T> 
TextOutput& operator<<(TextOutput& to, const wp<T>& val);



#undef COMPARE

#undef COMPARE_WEAK



// ---------------------------------------------------------------------------

// No user serviceable parts below here.