clover: Tidy up resource::mapping.

Tested-by: Tom Stellard <thomas.stellard@amd.com>

diff --git a/src/gallium/state_trackers/clover/core/resource.cpp b/src/gallium/state_trackers/clover/core/resource.cpp
index c503c86d7716786e76929970038d8cfaf2b80288..7e6b51f7c62236e25a73766970b821129eeabab5 100644 (file)
--- a/src/gallium/state_trackers/clover/core/resource.cpp
+++ b/src/gallium/state_trackers/clover/core/resource.cpp
@@ -71,7 +71,7 @@ resource::add_map(command_queue &q, cl_map_flags flags, bool blocking,
 
 void
 resource::del_map(void *p) {
-   erase_if([&](mapping &m) {
+   erase_if([&](const mapping &m) {
          return static_cast<void *>(m) == p;
       }, maps);
 }
@@ -185,8 +185,9 @@ mapping::mapping(command_queue &q, resource &r,
 
 mapping::mapping(mapping &&m) :
    pctx(m.pctx), pxfer(m.pxfer), p(m.p) {
-   m.p = NULL;
+   m.pctx = NULL;
    m.pxfer = NULL;
+   m.p = NULL;
 }
 
 mapping::~mapping() {

diff --git a/src/gallium/state_trackers/clover/core/resource.hpp b/src/gallium/state_trackers/clover/core/resource.hpp
index f447088055a234a865738a5aaa35c93ca77e4ae2..7a6a3f19a239721a19d6b81e70164662613ec9ae 100644 (file)
--- a/src/gallium/state_trackers/clover/core/resource.hpp
+++ b/src/gallium/state_trackers/clover/core/resource.hpp
@@ -62,7 +62,7 @@ namespace clover {
       friend class kernel;
 
    protected:
-      resource(device &dev, clover::memory_obj &obj);
+      resource(device &dev, memory_obj &obj);
 
       pipe_sampler_view *bind_sampler_view(command_queue &q);
       void unbind_sampler_view(command_queue &q,
@@ -86,8 +86,7 @@ namespace clover {
    public:
       root_resource(device &dev, memory_obj &obj,
                     command_queue &q, const std::string &data);
-      root_resource(device &dev, memory_obj &obj,
-                    root_resource &r);
+      root_resource(device &dev, memory_obj &obj, root_resource &r);
       virtual ~root_resource();
    };
 
@@ -109,19 +108,17 @@ namespace clover {
       mapping(command_queue &q, resource &r, cl_map_flags flags,
               bool blocking, const resource::vector &origin,
               const resource::vector &region);
-      mapping(const mapping &m) = delete;
       mapping(mapping &&m);
       ~mapping();
 
       mapping &
       operator=(mapping m);
 
-      operator void *() {
-         return p;
-      }
+      mapping(const mapping &m) = delete;
 
-      operator char *() {
-         return (char *)p;
+      template<typename T>
+      operator T *() const {
+         return (T *)p;
       }
 
    private: