Re: librados.hpp fix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux