do_postgres 0.10.7-java → 0.10.8-java

data/ChangeLog.markdown

@@ -1,3 +1,8 @@
+## 0.10.8 2012-02-10
+
+* Ruby 1.9.3 compatibility on Windows
+* Fix crash issue when reading a reader twice
+
 ## 0.10.7 2011-10-13
 
 * Ruby 1.9.3 compatibility

data/ext/do_postgres/do_common.c

@@ -53,9 +53,10 @@ VALUE data_objects_const_get(VALUE scope, const char *constant) {
 void data_objects_debug(VALUE connection, VALUE string, struct timeval *start) {
   struct timeval stop;
   VALUE message;
+  do_int64 duration;
 
   gettimeofday(&stop, NULL);
-  do_int64 duration = (stop.tv_sec - start->tv_sec) * 1000000 + stop.tv_usec - start->tv_usec;
+  duration = (stop.tv_sec - start->tv_sec) * 1000000 + stop.tv_usec - start->tv_usec;
 
   message = rb_funcall(cDO_Logger_Message, ID_NEW, 3, string, rb_time_new(start->tv_sec, start->tv_usec), INT2NUM(duration));
 
@@ -65,6 +66,7 @@ void data_objects_debug(VALUE connection, VALUE string, struct timeval *start) {
 void data_objects_raise_error(VALUE self, const struct errcodes *errors, int errnum, const char *message, VALUE query, VALUE state) {
   const char *exception_type = "SQLError";
   const struct errcodes *e;
+  VALUE uri, exception;
 
   for (e = errors; e->error_name; e++) {
     if (e->error_no == errnum) {
@@ -74,9 +76,9 @@ void data_objects_raise_error(VALUE self, const struct errcodes *errors, int err
     }
   }
 
-  VALUE uri = rb_funcall(rb_iv_get(self, "@connection"), rb_intern("to_s"), 0);
+  uri = rb_funcall(rb_iv_get(self, "@connection"), rb_intern("to_s"), 0);
 
-  VALUE exception = rb_funcall(
+  exception = rb_funcall(
     data_objects_const_get(mDO, exception_type),
     ID_NEW,
     5,
@@ -116,8 +118,8 @@ void data_objects_assert_file_exists(char *file, const char *message) {
 }
 
 VALUE data_objects_build_query_from_args(VALUE klass, int count, VALUE *args) {
-  VALUE array = rb_ary_new();
   int i;
+  VALUE array = rb_ary_new();
 
   for (i = 0; i < count; i++) {
     rb_ary_push(array, args[i]);
@@ -176,8 +178,6 @@ VALUE data_objects_timezone_to_offset(int hour_offset, int minute_offset) {
 VALUE data_objects_parse_date(const char *date) {
   static char const *const _fmt_date = "%4d-%2d-%2d";
   int year = 0, month = 0, day = 0;
-  int jd, ajd;
-  VALUE rational;
 
   switch (sscanf(date, _fmt_date, &year, &month, &day)) {
     case 0:
@@ -185,16 +185,7 @@ VALUE data_objects_parse_date(const char *date) {
       return Qnil;
   }
 
-#ifdef HAVE_NO_DATETIME_NEWBANG
   return rb_funcall(rb_cDate, ID_NEW, 3, INT2NUM(year), INT2NUM(month), INT2NUM(day));
-#else
-
-  jd       = data_objects_jd_from_date(year, month, day);
-  ajd      = (jd * 2) - 1;        // Math from Date.jd_to_ajd
-  rational = rb_funcall(rb_mKernel, ID_RATIONAL, 2, INT2NUM(ajd), INT2NUM(2));
-
-  return rb_funcall(rb_cDate, ID_NEW_DATE, 3, rational, INT2NUM(0), INT2NUM(2299161));
-#endif
 }
 
 VALUE data_objects_parse_time(const char *date) {
@@ -224,10 +215,9 @@ VALUE data_objects_parse_date_time(const char *date) {
   int tokens_read;
   const char *fmt_datetime;
 
-  VALUE ajd, offset;
+  VALUE offset;
 
-  int year, month, day, hour, min, sec, hour_offset, minute_offset, jd;
-  do_int64 num, den;
+  int year, month, day, hour, min, sec, hour_offset, minute_offset;
 
   struct tm timeinfo;
   time_t target_time;
@@ -304,40 +294,9 @@ VALUE data_objects_parse_date_time(const char *date) {
       rb_raise(eDataError, "Couldn't parse date: %s", date);
   }
 
-#ifdef HAVE_NO_DATETIME_NEWBANG
   offset = data_objects_timezone_to_offset(hour_offset, minute_offset);
   return rb_funcall(rb_cDateTime, ID_NEW, 7, INT2NUM(year), INT2NUM(month), INT2NUM(day),
                                              INT2NUM(hour), INT2NUM(min), INT2NUM(sec), offset);
-#else
-  jd = data_objects_jd_from_date(year, month, day);
-
-  /*
-   * Generate ajd with fractional days for the time.
-   * Extracted from Date#jd_to_ajd, Date#day_fraction_to_time, and Rational#+ and #-.
-   *
-   * TODO: These are 64bit numbers; is reduce() really necessary?
-   */
-
-  num = (hour * 1440) + (min * 24);
-  num -= (hour_offset * 1440) + (minute_offset * 24);
-  den = (24 * 1440);
-  data_objects_reduce(&num, &den);
-
-  num = (num * 86400) + (sec * den);
-  den = den * 86400;
-  data_objects_reduce(&num, &den);
-
-  num += jd * den;
-
-  num = (num * 2) - den;
-  den *= 2;
-  data_objects_reduce(&num, &den);
-
-  ajd = rb_funcall(rb_mKernel, ID_RATIONAL, 2, rb_ull2inum(num), rb_ull2inum(den));
-  offset = data_objects_timezone_to_offset(hour_offset, minute_offset);
-
-  return rb_funcall(rb_cDateTime, ID_NEW_DATE, 3, ajd, offset, INT2NUM(2299161));
-#endif
 }
 
 VALUE data_objects_cConnection_character_set(VALUE self) {
@@ -371,17 +330,15 @@ VALUE data_objects_cConnection_quote_date(VALUE self, VALUE value) {
  * into Ruby-strings so we can easily typecast later
  */
 VALUE data_objects_cCommand_set_types(int argc, VALUE *argv, VALUE self) {
+  VALUE entry, sub_entry;
+  int i, j;
   VALUE type_strings = rb_ary_new();
   VALUE array = rb_ary_new();
 
-  int i, j;
-
   for (i = 0; i < argc; i++) {
     rb_ary_push(array, argv[i]);
   }
 
-  VALUE entry, sub_entry;
-
   for (i = 0; i < RARRAY_LEN(array); i++) {
     entry = rb_ary_entry(array, i);
 

data/ext/do_postgres/do_postgres.c

@@ -610,7 +610,15 @@ VALUE do_postgres_cReader_close(VALUE self) {
 }
 
 VALUE do_postgres_cReader_next(VALUE self) {
-  PGresult *reader = DATA_PTR(rb_iv_get(self, "@reader"));
+
+  VALUE reader = rb_iv_get(self, "@reader");
+
+  if(reader == Qnil) {
+    rb_raise(eConnectionError, "This result set has already been closed.");
+    return Qfalse;
+  }
+
+  PGresult *pg_reader = DATA_PTR(reader);
 
   int row_count = NUM2INT(rb_iv_get(self, "@row_count"));
   int field_count = NUM2INT(rb_iv_get(self, "@field_count"));
@@ -642,8 +650,8 @@ VALUE do_postgres_cReader_next(VALUE self) {
     field_type = rb_ary_entry(field_types, i);
 
     // Always return nil if the value returned from Postgres is null
-    if (!PQgetisnull(reader, position, i)) {
-      value = do_postgres_typecast(PQgetvalue(reader, position, i), PQgetlength(reader, position, i), field_type, enc);
+    if (!PQgetisnull(pg_reader, position, i)) {
+      value = do_postgres_typecast(PQgetvalue(pg_reader, position, i), PQgetlength(pg_reader, position, i), field_type, enc);
     }
     else {
       value = Qnil;

data/lib/do_postgres/version.rb

@@ -1,5 +1,5 @@
 module DataObjects
   module Postgres
-    VERSION = '0.10.7'
+    VERSION = '0.10.8'
   end
 end

data/spec/typecast/date_spec.rb

@@ -6,4 +6,28 @@ require 'data_objects/spec/shared/typecast/date_spec'
 describe 'DataObjects::Postgres with Date' do
   it_should_behave_like 'supporting Date'
   it_should_behave_like 'supporting Date autocasting'
+
+  describe 'exotic dates' do
+
+    before do
+      @connection = DataObjects::Connection.new(CONFIG.uri)
+      @connection.create_command("INSERT INTO widgets (release_date) VALUES ('0001-01-01')").execute_non_query
+
+      @command = @connection.create_command("SELECT release_date FROM widgets WHERE release_date = '0001-01-01'")
+      @reader = @command.execute_reader
+      @reader.next!
+      @values = @reader.values
+    end
+
+    after do
+      @reader.close
+      @connection.close
+    end
+
+    it 'should return the number of created rows' do
+      @values.first.should == Date.civil(1, 1, 1)
+    end
+
+  end
+
 end

data/tasks/compile.rake

@@ -70,7 +70,7 @@ begin
     ext.classpath = '../do_jdbc/lib/do_jdbc_internal.jar'
     ext.java_compiling do |gem|
       gem.add_dependency 'jdbc-postgres', '>=8.2'
-      gem.add_dependency 'do_jdbc',       '0.10.7'
+      gem.add_dependency 'do_jdbc',       '0.10.8'
     end
   end
 rescue LoadError

data/tasks/spec.rake

@@ -2,7 +2,6 @@ require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec => [:clean, :compile]) do |spec|
   spec.pattern      = './spec/**/*_spec.rb'
-  spec.skip_bundler = true
 end
 
 RSpec::Core::RakeTask.new(:rcov => [:clean, :compile]) do |rcov|

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: do_postgres
 version: !ruby/object:Gem::Version 
-  hash: 57
+  hash: 39
   prerelease: 
   segments: 
   - 0
   - 10
-  - 7
-  version: 0.10.7
+  - 8
+  version: 0.10.8
 platform: java
 authors: 
 - Dirkjan Bussink
@@ -18,23 +18,23 @@ cert_chain: []
 date: 2011-03-29 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  version_requirements: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 57
+        hash: 39
         segments: 
         - 0
         - 10
-        - 7
-        version: 0.10.7
-  version_requirements: *id001
+        - 8
+        version: 0.10.8
   name: data_objects
   prerelease: false
   type: :runtime
+  requirement: *id001
 - !ruby/object:Gem::Dependency 
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -44,12 +44,12 @@ dependencies:
         - 2
         - 5
         version: "2.5"
-  version_requirements: *id002
   name: rspec
   prerelease: false
   type: :development
+  requirement: *id002
 - !ruby/object:Gem::Dependency 
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -59,12 +59,12 @@ dependencies:
         - 0
         - 7
         version: "0.7"
-  version_requirements: *id003
   name: rake-compiler
   prerelease: false
   type: :development
+  requirement: *id003
 - !ruby/object:Gem::Dependency 
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -74,26 +74,26 @@ dependencies:
         - 8
         - 2
         version: "8.2"
-  version_requirements: *id004
   name: jdbc-postgres
   prerelease: false
   type: :runtime
+  requirement: *id004
 - !ruby/object:Gem::Dependency 
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 57
+        hash: 39
         segments: 
         - 0
         - 10
-        - 7
-        version: 0.10.7
-  version_requirements: *id005
+        - 8
+        version: 0.10.8
   name: do_jdbc
   prerelease: false
   type: :runtime
+  requirement: *id005
 description: Implements the DataObjects API for PostgreSQL
 email: d.bussink@gmail.com
 executables: []
@@ -175,7 +175,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: dorb
-rubygems_version: 1.8.6
+rubygems_version: 1.8.14
 signing_key: 
 specification_version: 3
 summary: DataObjects PostgreSQL Driver