librados.hpp fix

librados.hpp fix

[Takuya ASADA]

Hi,

I just realized librados.hpp could compile only if "using namespace
std;" is declared before "#include <rados/librados.hpp>", because
"vector" used without std namespace, and I think it's wrong.
It can fix following patch.

syuu

diff --git a/src/include/librados.hpp b/src/include/librados.hpp
index b0d3622..06fa3b2 100644
--- a/src/include/librados.hpp
+++ b/src/include/librados.hpp
@@ -38,7 +38,7 @@ namespace librados {

   struct SnapContext {
     snap_t seq;
-    vector<snap_t> snaps;
+    std::vector<snap_t> snaps;
   };


@@ -57,7 +57,7 @@ public:
   int lookup_pool(const char *name);

   void set_snap(pool_t pool, snap_t seq);
-  int set_snap_context(pool_t pool, snap_t seq, vector<snap_t>& snaps);
+  int set_snap_context(pool_t pool, snap_t seq, std::vector<snap_t>& snaps);


   int create(pool_t pool, const std::string& oid, bool exclusive);

Re: librados.hpp fix

[Sage Weil]

Thanks, applied this.  And fixed the acconfig.h #include.  Pushed to 
the testing and unstable branches.

sage

On Tue, 3 Aug 2010, Takuya ASADA wrote:

> Hi,
> 
> I just realized librados.hpp could compile only if "using namespace
> std;" is declared before "#include <rados/librados.hpp>", because
> "vector" used without std namespace, and I think it's wrong.
> It can fix following patch.
> 
> syuu
> 
> diff --git a/src/include/librados.hpp b/src/include/librados.hpp
> index b0d3622..06fa3b2 100644
> --- a/src/include/librados.hpp
> +++ b/src/include/librados.hpp
> @@ -38,7 +38,7 @@ namespace librados {
> 
>    struct SnapContext {
>      snap_t seq;
> -    vector<snap_t> snaps;
> +    std::vector<snap_t> snaps;
>    };
> 
> 
> @@ -57,7 +57,7 @@ public:
>    int lookup_pool(const char *name);
> 
>    void set_snap(pool_t pool, snap_t seq);
> -  int set_snap_context(pool_t pool, snap_t seq, vector<snap_t>& snaps);
> +  int set_snap_context(pool_t pool, snap_t seq, std::vector<snap_t>& snaps);
> 
> 
>    int create(pool_t pool, const std::string& oid, bool exclusive);
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
>