Pointless hash function in simple store

The implementation of XmlNode::hash() is:

  uint32_t hash(long timezone = 0, const XQPCollator* aCollation = 0) const
  {
    XmlNode* node = const_cast<XmlNode*>(this);
    return hashfun::h32((void*)(&node), sizeof(node), FNV_32_INIT);
  }

Computing the hash of a pointer is pointless -- just cast the pointer:

  uint32_t hash(long timezone = 0, const XQPCollator* aCollation = 0) const
  {
    return reinterpret_cast<uint32_t>( this );
  }

Indeed, the std::hash() function in C++11 for T* does exactly this.