Sorry, wrong patch attached. Here comes the updated version. Best regards. On Mon, Jun 24, 2013 at 4:36 PM, christophe courtaut <christophe.courtaut@xxxxxxxxx> wrote: > Here come a new patch. > > I do not generate an encoder/decoder using WRITE_RAW_ENCODER anymore. > > Instead i added a encoder/decoder for time_t using a __u64. > > On Thu, Jun 20, 2013 at 10:44 PM, Yehuda Sadeh <yehuda@xxxxxxxxxxx> wrote: >> On Thu, Jun 20, 2013 at 6:57 AM, christophe courtaut >> <christophe.courtaut@xxxxxxxxx> wrote: >>> On the rgw-wip-geo-2 branch, the encode decode function are called >>> with time_t now, >>> but there is no encode/decode function created for time_t type. >>> >>> http://tracker.ceph.com/issues/5402 refs #5402 >>> >>> This patch adds a WRITE_RAW_ENCODER to support this type. >>> >>> -- >>> Christophe Courtaut >> >> >> The problem here is that we can't use the generic encoder / decoder as >> time_t may have different size on different architectures. We should >> just have a fixed size encoding for it (probably 64 bit albeit the >> waste). >> >> Thanks, >> Yehuda > > > > -- > Christophe Courtaut -- Christophe Courtaut
Attachment:
0001-rgw-Added-an-encoder-for-time_t-to-fix-compilation.patch
Description: Binary data
