On 10.05.2011 11:14, Branko Čibej wrote:
On 10.05.2011 10:34, Stefan Fuhrmann wrote:
On 09.05.2011 18:43, Philip Martin wrote:
stef...@apache.org writes:

Author: stefan2
Date: Mon May  9 16:28:16 2011
New Revision: 1101091
    (svn_cache__membuffer_cache_create): formatting fixes; add
missing cast
-  c = apr_palloc(pool, segment_count * sizeof(*c));
+  c = (svn_membuffer_t *)apr_palloc(pool, segment_count * sizeof(*c));
Why is that cast necessary?

The Microsoft compiler issued a warning here and
the explicit cast silences it. Maybe, it's a C99 thing?
Can't be, no Microsoft compiler has ever pretended to implement C99 ...
besides, the rules for void* did not change between C90 and C99.
So, I found out what happened: The *IDE* (editor) complained
about the assignment and thus I "fixed" it without checking the
compiler log.

The compiler is happy without the cast and r1101616.

-- Stefan^2.

Reply via email to