[ruby-oci8-commit] [301] branches/ruby-oci8-1.0: * ext/oci8/const.c, ext/oci8/handle.c: suppress compilation warnings.

[nobody at rubyforge.org]

Revision: 301
Author:   kubo
Date:     2008-12-30 04:03:05 -0500 (Tue, 30 Dec 2008)

Log Message:
-----------
* ext/oci8/const.c, ext/oci8/handle.c: suppress compilation warnings.
    (contributed by Daniel Berger)

Modified Paths:
--------------
    branches/ruby-oci8-1.0/ChangeLog
    branches/ruby-oci8-1.0/ext/oci8/const.c
    branches/ruby-oci8-1.0/ext/oci8/handle.c

Modified: branches/ruby-oci8-1.0/ChangeLog
===================================================================
--- branches/ruby-oci8-1.0/ChangeLog	2008-12-30 07:23:14 UTC (rev 300)
+++ branches/ruby-oci8-1.0/ChangeLog	2008-12-30 09:03:05 UTC (rev 301)
@@ -1,4 +1,8 @@
 2008-12-30  KUBO Takehiro  <kubo at jiubao.org>
+	* ext/oci8/const.c, ext/oci8/handle.c: suppress compilation warnings.
+	    (contributed by Daniel Berger)
+
+2008-12-30  KUBO Takehiro  <kubo at jiubao.org>
 	* ext/oci8/oraconf.rb: add code to check the cpu type of Mac OS X
 	    libclntsh.dylib.
 

Modified: branches/ruby-oci8-1.0/ext/oci8/const.c
===================================================================
--- branches/ruby-oci8-1.0/ext/oci8/const.c	2008-12-30 07:23:14 UTC (rev 300)
+++ branches/ruby-oci8-1.0/ext/oci8/const.c	2008-12-30 09:03:05 UTC (rev 301)
@@ -45,7 +45,7 @@
 static VALUE oci8_make_names(oci8_names_t *names, size_t size)
 {
   volatile VALUE ary;
-  int i;
+  unsigned int i;
 
   ary = rb_ary_new();
   for (i = 0;i < size;i++)
@@ -55,7 +55,7 @@
 
 void  Init_oci8_const(void)
 {
-  int i;
+  unsigned int i;
 
   oci8_id_code = rb_intern("code");
   oci8_id_define_array = rb_intern("define_array");

Modified: branches/ruby-oci8-1.0/ext/oci8/handle.c
===================================================================
--- branches/ruby-oci8-1.0/ext/oci8/handle.c	2008-12-30 07:23:14 UTC (rev 300)
+++ branches/ruby-oci8-1.0/ext/oci8/handle.c	2008-12-30 09:03:05 UTC (rev 301)
@@ -13,7 +13,7 @@
 
 static void oci8_handle_do_free(oci8_handle_t *h)
 {
-  int i;
+  unsigned int i;
   if (h->type == 0) {
     return;
   }
@@ -104,7 +104,7 @@
 static void oci8_handle_mark(oci8_handle_t *h)
 {
   oci8_bind_handle_t *bh;
-  int i;
+  unsigned int i;
 
   switch (h->type) {
   case OCI_HTYPE_SVCCTX:
@@ -221,8 +221,8 @@
 
 void oci8_link(oci8_handle_t *parent, oci8_handle_t *child)
 {
-  int i;
-  int new_size;
+  unsigned int i;
+  int new_size, n;
 
   if (parent == NULL)
     return;
@@ -238,8 +238,8 @@
   new_size = parent->size + CHILDREN_ARRAY_GROW_SIZE;
   parent->children = xrealloc(parent->children, sizeof(oci8_handle_t *) * new_size);
   parent->children[parent->size] = child;
-  for (i = parent->size + 1;i < new_size;i++) {
-    parent->children[i] = NULL;
+  for (n = parent->size + 1;n < new_size;n++) {
+    parent->children[n] = NULL;
   }
   parent->size = new_size;
   return;
@@ -248,7 +248,7 @@
 void oci8_unlink(oci8_handle_t *self)
 {
   oci8_handle_t *parent = self->parent;
-  int i;
+  unsigned int i;
 
   if (self->parent == NULL)
     return;