What am i doing wrong

Question

I have the following code. I need B class to have a min priority queue of AToTime objects. AToTime have operator>, and yet i receive error telling me than there is no operator> matching the operands...

#include <queue>
#include <functional>

using namespace std; 

class B{
//public functions
public:

    B();

    virtual ~B();

//private members
private:

    log4cxx::LoggerPtr      m_logger;

    class AToTime 
    {
    //public functions
    public:
        AToTime(const ACE_Time_Value& time, const APtr a) : m_time(time), m_a(a){}

        bool operator >(const AToTime& other)
        {
            return m_time > other.m_time;
        }

    //public members - no point using any private members here
    public:
        ACE_Time_Value      m_time;
        APtr                m_a;
    };


    priority_queue<AToTime, vector<AToTime>, greater<AToTime> > m_myMinHeap;
};

Answer 1

    bool operator >(const AToTime& other)

It should be a const function.

    bool operator >(const AToTime& other) const 

Answer 2

Kenny's answer already shows you how to make this work.

Note that I would prefer to implement binary operators which treat their operands equally (they're not modifying them) as free functions:

inline bool operator>(const AToTime& khs, const AToTime& rhs)
{
  return lhs.m_time > rhs.m_time;
}

Further, usually users expect all relational operators to be present if one of them is there. Since the std library mostly wants operator<, except for equality I'd implement the others on top of operator<:

inline bool operator<(const AToTime& khs, const AToTime& rhs)
{return lhs.m_time < rhs.m_time;}

inline bool operator>(const AToTime& khs, const AToTime& rhs)
{return rhs < lhs;}

inline bool operator<=(const AToTime& khs, const AToTime& rhs)
{return !(lhs > rhs);}

inline bool operator>=(const AToTime& khs, const AToTime& rhs)
{return !(lhs < rhs);}

inline bool operator==(const AToTime& khs, const AToTime& rhs)
{return lhs.m_time == rhs.m_time;}

inline bool operator!=(const AToTime& khs, const AToTime& rhs)
{return !(lhs.m_time == rhs.m_time);}