From f9c014d008e42eab79cffd6441147cbc95963cf5 Mon Sep 17 00:00:00 2001 From: nobu Date: Wed, 18 Jul 2012 07:44:17 +0000 Subject: empty_p: optimize and warnings * ext/dbm/dbm.c (fdbm_empty_p): not empty if first key exists. suppress warnngs by gcc 4.7 -Wstrict-overflow. * ext/sdbm/init.c (fsdbm_empty_p): ditto. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@36438 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ext/sdbm/init.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'ext/sdbm') diff --git a/ext/sdbm/init.c b/ext/sdbm/init.c index 65e2ded07b..1795024ff9 100644 --- a/ext/sdbm/init.c +++ b/ext/sdbm/init.c @@ -714,21 +714,20 @@ fsdbm_empty_p(VALUE obj) datum key; struct dbmdata *dbmp; DBM *dbm; - int i = 0; GetDBM(obj, dbmp); if (dbmp->di_size < 0) { dbm = dbmp->di_dbm; for (key = sdbm_firstkey(dbm); key.dptr; key = sdbm_nextkey(dbm)) { - i++; + return Qfalse; } } else { - i = dbmp->di_size; + if (!dbmp->di_size) + return Qfalse; } - if (i == 0) return Qtrue; - return Qfalse; + return Qtrue; } /* -- cgit v1.2.3