strptime 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d931cdcc8f8fd8337c8ec9a266dbaed766095aa9
-  data.tar.gz: c4681f0b6c9dd96f01fc8a4e5736bf1ca2c71199
+  metadata.gz: d0aeb413a53bd63240a359b0343bf5e3d3204bdf
+  data.tar.gz: 358990f23e0e9885c2e20f1dc2301c772bb3bb70
 SHA512:
-  metadata.gz: 12f64e3d51c7828fb59ef9cb6369cca85c248504b8b5ec695d3325ae8ca9bd91a81ddcf66b77544f52a71c9535ba34580dce52992ecb5c222f2cf8da691a4f1d
-  data.tar.gz: e55daa93f057600590636b54743ae5b248314afe10bb514c7537ed9204e4b67ece78352c261edc5c53cf363159cb37eeba0642fd053f61eeb73fb898fe34398e
+  metadata.gz: 10d2cb9ec99244fd81d54de6d8a5b6e0a8fb4218dbd22716f95c2df586eedbb50309fba82604f61aaa0232f523145bd4e115652e5de373828c5aee7ce014d5ef
+  data.tar.gz: a57fe751a496f587dda41fc500c49a2433a46b744d8a79819804b71204d805d2796a2e3024ffad54e3aa3572ac639f6ad8dcd734385e7915c938a23540ef77e0

data/.travis.yml

@@ -3,7 +3,7 @@ os:
   - linux
   - osx
 rvm:
-  - 2.0.0-p647
+  - 2.0.0
   - 2.1
   - 2.2.3
   - 2.2.3-clang

data/appveyor.yml

@@ -4,10 +4,8 @@ install:
   - ruby --version
   - gem --version
   - bundle install
-build: off
-test_script:
+build_script:
   - bundle exec rake -rdevkit
-
 environment:
   matrix:
     - ruby_version: "200"

data/ext/strptime/extconf.rb

@@ -1,5 +1,6 @@
 require "mkmf"
 
-have_func 'timegm', 'time.h' or 'timegm(3) is required'
+have_func('rb_timespec_now')
+have_func('rb_time_timespec_new')
 
 create_makefile("strptime/strptime")

data/ext/strptime/ruby_time.c

@@ -20,7 +20,8 @@
 typedef uint64_t WIDEVALUE;
 typedef WIDEVALUE wideval_t;
 
-#ifdef PACKED_STRUCT_UNALIGNED
+#ifndef HAVE_RB_TIME_TIMESPEC_NEW
+# if defined(PACKED_STRUCT_UNALIGNED) /* 2.2 */
 PACKED_STRUCT_UNALIGNED(struct vtm {
     VALUE year;	/* 2000 for example.  Integer. */
     VALUE subsecx;     /* 0 <= subsecx < TIME_SCALE.  possibly Rational. */
@@ -41,7 +42,7 @@ PACKED_STRUCT_UNALIGNED(struct time_object {
     uint8_t gmt : 3; /* 0:utc 1:localtime 2:fixoff 3:init */
     uint8_t tm_got : 1;
 });
-#else
+# else /* 2.0.0~2.1 */
 struct vtm {
     VALUE year; /* 2000 for example.  Integer. */
     int mon; /* 1..12 */
@@ -62,15 +63,13 @@ struct time_object {
     int gmt; /* 0:utc 1:localtime 2:fixoff */
     int tm_got;
 };
-#endif
+# endif
 
 VALUE
-rb_time_succ(VALUE time);
-VALUE
-rbtime_timespec_new(const struct timespec *ts, int offset)
+rb_time_timespec_new(const struct timespec *ts, int offset)
 {
     VALUE obj = rb_time_nano_new(ts->tv_sec, ts->tv_nsec);
-    if (offset) {
+    if (-86400 < offset && offset <  86400) { /* fixoff */
 	struct time_object *tobj;
 	tobj = DATA_PTR(obj);
 	tobj->tm_got = 0;
@@ -78,12 +77,25 @@ rbtime_timespec_new(const struct timespec *ts, int offset)
 	tobj->vtm.utc_offset = INT2FIX(offset);
 	tobj->vtm.zone = NULL;
     }
+    else if (offset == INT_MAX) { /* localtime */
+    }
+    else if (offset == INT_MAX-1) { /* UTC */
+	struct time_object *tobj;
+	tobj = DATA_PTR(obj);
+	tobj->tm_got = 0;
+	tobj->gmt = 1;
+    }
+    else {
+	rb_raise(rb_eArgError, "utc_offset out of range");
+    }
+
     return obj;
 }
+#endif
 
-/* timespec_now */
+#ifndef RB_TIMESPEC_NOW
 void
-timespec_now(struct timespec *ts)
+rb_timespec_now(struct timespec *ts)
 {
 #ifdef HAVE_CLOCK_GETTIME
     if (clock_gettime(CLOCK_REALTIME, ts) == -1) {
@@ -100,6 +112,7 @@ timespec_now(struct timespec *ts)
     }
 #endif
 }
+#endif
 
 /* localtime_with_gmtoff_zone */
 #ifdef HAVE_GMTIME_R

data/ext/strptime/strptime.c

@@ -365,7 +365,7 @@ strptime_exec0(void **pc, const char *fmt, const char *str, size_t slen,
 	const char *p0 = str + si;
 	int r;
 	size_t len;
-	if (*p0 == 'z') {
+	if (*p0 == 'z' || *p0 == 'Z') {
 	    gmtoff = 0;
 	    ADD_PC(1);
 	    END_INSN(z)
@@ -420,9 +420,10 @@ strptime_exec0(void **pc, const char *fmt, const char *str, size_t slen,
 	struct timespec ts;
 	struct tm tm;
 	time_t t;
+	int gmt = gmtoff >= INT_MAX-1 ? INT_MAX-gmtoff : 2;
 
 	/* get current time with timezone */
-	timespec_now(&ts);
+	rb_timespec_now(&ts);
 	{
 	    static time_t ct;
 	    static struct tm ctm;
@@ -436,6 +437,9 @@ strptime_exec0(void **pc, const char *fmt, const char *str, size_t slen,
 	    if (gmtoff == INT_MAX) {
 		gmtoff = localoff;
 	    }
+	    else if (gmtoff == INT_MAX-1) {
+		gmtoff = 0;
+	    }
 	    if (gmtoff != ctmoff) {
 		tm_add_offset(&ctm, gmtoff - ctmoff);
 		ctmoff = gmtoff;
@@ -469,7 +473,8 @@ strptime_exec0(void **pc, const char *fmt, const char *str, size_t slen,
 	    if (sec != -1) tm.tm_sec = sec;
 	}
 
-	t = timegm_noleapsecond(&tm) - gmtoff;
+	t = timegm_noleapsecond(&tm);
+	if (gmt != 1) t -= gmtoff;
 	tsp->tv_sec = t;
 	tsp->tv_nsec = nsec;
 	*gmtoffp = gmtoff;
@@ -658,32 +663,6 @@ strptime_init_copy(VALUE copy, VALUE self)
     return copy;
 }
 
-typedef uint64_t WIDEVALUE;
-typedef WIDEVALUE wideval_t;
-#ifndef PACKED_STRUCT_UNALIGNED
-#define PACKED_STRUCT_UNALIGNED(x) x
-#endif
-PACKED_STRUCT_UNALIGNED(struct vtm {
-    VALUE year;	/* 2000 for example.  Integer. */
-    VALUE subsecx;     /* 0 <= subsecx < TIME_SCALE.  possibly Rational. */
-    VALUE utc_offset;  /* -3600 as -01:00 for example.  possibly Rational. */
-    const char *zone;  /* "JST", "EST", "EDT", etc. */
-    uint16_t yday : 9; /* 1..366 */
-    uint8_t mon : 4;   /* 1..12 */
-    uint8_t mday : 5;  /* 1..31 */
-    uint8_t hour : 5;  /* 0..23 */
-    uint8_t min : 6;   /* 0..59 */
-    uint8_t sec : 6;   /* 0..60 */
-    uint8_t wday : 3;  /* 0:Sunday, 1:Monday, ..., 6:Saturday 7:init */
-    uint8_t isdst : 2; /* 0:StandardTime 1:DayLightSavingTime 3:init */
-});
-PACKED_STRUCT_UNALIGNED(struct time_object {
-    wideval_t timew; /* time_t value * TIME_SCALE.  possibly Rational. */
-    struct vtm vtm;
-    uint8_t gmt : 3; /* 0:utc 1:localtime 2:fixoff 3:init */
-    uint8_t tm_got : 1;
-});
-
 /*
  * @overload exec(str)
  *   @param str [String] string to parse
@@ -696,15 +675,15 @@ static VALUE
 strptime_exec(VALUE self, VALUE str)
 {
     struct strptime_object *tobj;
-    int r, gmtoff = 0;
+    int r, gmtoff = INT_MAX;
+    struct timespec ts;
     StringValue(str);
     GetStrptimeval(self, tobj);
-    struct timespec ts;
 
     r = strptime_exec0(tobj->isns, RSTRING_PTR(tobj->fmt), RSTRING_PTR(str),
 		       RSTRING_LEN(str), &ts, &gmtoff);
     if (r) rb_raise(rb_eArgError, "string doesn't match");
-    return rbtime_timespec_new(&ts, gmtoff);
+    return rb_time_timespec_new(&ts, gmtoff);
 }
 
 /*
@@ -719,7 +698,7 @@ strptime_execi(VALUE self, VALUE str)
 {
     struct strptime_object *tobj;
     struct timespec ts;
-    int r, gmtoff = 0;
+    int r, gmtoff = INT_MAX;
     StringValue(str);
     GetStrptimeval(self, tobj);
 

data/ext/strptime/strptime.h

@@ -2,9 +2,13 @@
 #define STRPTIME_H 1
 
 #include "ruby.h"
-VALUE rbtime_timespec_new(const struct timespec *ts, int offset);
+# ifndef HAVE_RB_TIME_TIMESPEC_NEW
+VALUE rb_time_timespec_new(const struct timespec *ts, int offset);
+# endif
 struct tm * localtime_with_gmtoff_zone(const time_t *t, struct tm *result, long *gmtoff, const char **zone);
-void timespec_now(struct timespec *ts);
+# ifndef HAVE_RB_TIMESPEC_NOW
+void rb_timespec_now(struct timespec *ts);
+# endif
 time_t timegm_noleapsecond(struct tm *tm);
 void tm_add_offset(struct tm *tm, long diff);
 

data/lib/strptime/version.rb

@@ -1,3 +1,3 @@
 class Strptime
-  VERSION = "0.1.5"
+  VERSION = "0.1.6"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: strptime
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - NARUSE, Yui
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-11-05 00:00:00.000000000 Z
+date: 2015-12-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -140,7 +140,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: a fast strptime engine.