[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r927344 - in /subversion/trunk/subversion/libsvn_wc: upgrade.c wc_db.c

From: Greg Stein <gstein_at_gmail.com>
Date: Thu, 25 Mar 2010 09:26:00 -0400

On Thu, Mar 25, 2010 at 07:03, <julianfoad_at_apache.org> wrote:
>...
> * subversion/libsvn_wc/wc_db.c
>  (insert_base_node, svn_wc__db_op_set_last_mod_time): Pass integers to
>    svn_sqlite__bindf as 64-bit values, as required by the latter.  (The
>    first one here is typedef'd to 'long int' which may not be the same on
>    all architectures.  The second one here is typedef'd to the right type
>    in my version of APR but it's obscure to rely on that.)

It is NOT obscure. An apr_time_t is *defined* as a 64-bit integer, as
microseconds since epoch. The entire apr_time interface revolves
around that specific definition. Thus, your cast because wholly
redundant and (for me) causes the exact opposite problem. "What? I
thought this was a time? What's the cast in there for? ... Oh. Damn
useless cast."

Please revert the time cast.

>...

Cheers,
-g
Received on 2010-03-25 14:26:30 CET

This is an archived mail posted to the Subversion Dev mailing list.