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

Review of sizeof usage

From: Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
Date: Tue, 11 Aug 2015 15:50:51 +0100

Hi all,

The bug that lead to r1694533 prompted me to grep
for all 2500+ uses of sizeof and to review them.

I only found one other real problem and fixed it in
r1694929. However, I noticed an inconsistency in the
way we use sizeof. In my opinion, we should take the
size of the created or processed variable instead of its
type, i.e.

  abc_t *v = apr_pcalloc(pool, sizeof(*v));
  apr_hash_set(hash, key, sizeof(*key), y);
  z = apr_hash_get(hash, key, sizeof(*key));

rather than

  abc_t *v = apr_pcalloc(pool, sizeof(abc_t));
  apr_hash_set(hash, key, sizeof(key_t), y);
  z = apr_hash_get(hash, key, sizeof(key_t));

I found and fixed about 50 occurrences. If nobody
objects, I'll commit them over the weekend.

-- Stefan^2.
Received on 2015-08-11 16:50:56 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.