oj 2.11.5 → 2.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8d1ef3e0a7644077283534c35c890016691f9592
-  data.tar.gz: 671cfd43d8f6b32f3d84a389326e090ca42fa2f2
+  metadata.gz: b4f6db0371ed698ce4c6002a7d12acfb79982191
+  data.tar.gz: 8255ac3b86ed139ebbb350aacf8a8b25cd4806d1
 SHA512:
-  metadata.gz: e443fc1c93a8399fa30e5bf48e10547bbf5baf7a5cd749a9e37b8fd234d4e8bb15dbc1942012f5c5cf7eff60064d5a37447e6a69f7beaea7bbdcb4dfea54bd0c
-  data.tar.gz: 323cb80c48f393bca6ffe4b11dae19af16ce88b2b6f2f5052264de1e3487fafd04ab61ce4daadb76eb1be5737a1fb13c47adec5572451f24ede11be8777be89a
+  metadata.gz: ee7a31844af7de0fcf4074146e963f17952cf396de304af36b7b04ad35a9e6904dc3ff93207cd728fbcb5102a5f4d8307942df9124cb4085b82c07ab3f0111c1
+  data.tar.gz: daa03c5114facec630dea16ce5925aa873a398e008383a6fc9323c3a468d6dec173167f1c071c036192564a3e9ec856e0d5b6b23222d21ff90aabf4e2900987b

data/README.md

@@ -26,19 +26,16 @@ Follow [@peterohler on Twitter](http://twitter.com/#!/peterohler) for announceme
 
 [![Build Status](https://secure.travis-ci.org/ohler55/oj.png?branch=master)](http://travis-ci.org/ohler55/oj)
 
-## Release 2.11.5
+## Future Release 2.12.0
 
- - Fixed issue with rails as_json not being called for Structs.
+ - String formats for UTC time are now explitly UTC instead of offset of
+   zero. This fixes a problem with pre-2.2.0 Rubies that automatically convert
+   zero offset times to local times.
 
- - Added support for anonymous Structs with object mode encoding. Note that this
-   will result in a new anonymous Struct for each instance.
-
-## Release 2.11.4
-
- - DateTime second encoding is now always a Rational to preserve accuracy.
-
- - Fixed bug in the Oj.load() callback feature that caused an endless loop when
-   a StringIO was used with a JSON that was a number.
+ - Added :unix_zone time_format option for formating numeric time. This option
+   is the same as the :unix time option but the UTC offset is included as an
+   exponent to the number time value. A value of 86400 is an indication of UTC
+   time.
 
 [Older release notes](http://www.ohler.com/dev/oj_misc/release_notes.html).
 

data/ext/oj/dump.c

@@ -80,7 +80,7 @@ static int	hash_cb_strict(VALUE key, VALUE value, Out out);
 static int	hash_cb_compat(VALUE key, VALUE value, Out out);
 static int	hash_cb_object(VALUE key, VALUE value, Out out);
 static void	dump_hash(VALUE obj, VALUE clas, int depth, int mode, Out out);
-static void	dump_time(VALUE obj, Out out);
+static void	dump_time(VALUE obj, Out out, int withZone);
 static void	dump_ruby_time(VALUE obj, Out out);
 static void	dump_xml_time(VALUE obj, Out out);
 static void	dump_data_strict(VALUE obj, Out out);
@@ -996,11 +996,11 @@ dump_hash(VALUE obj, VALUE clas, int depth, int mode, Out out) {
 }
 
 static void
-dump_time(VALUE obj, Out out) {
+dump_time(VALUE obj, Out out, int withZone) {
     char		buf[64];
     char		*b = buf + sizeof(buf) - 1;
     long		size;
-    char		*dot = b - 10;
+    char		*dot;
     int			neg = 0;
     long		one = 1000000000;
 #if HAS_RB_TIME_TIMESPEC
@@ -1016,6 +1016,31 @@ dump_time(VALUE obj, Out out) {
 #endif
 #endif
     
+    *b-- = '\0';
+    if (withZone) {
+	long	tzsecs = NUM2LONG(rb_funcall2(obj, oj_utc_offset_id, 0, 0));
+	int	zneg = (0 > tzsecs);
+
+	if (0 == tzsecs && Qtrue == rb_funcall2(obj, oj_utcq_id, 0, 0)) {
+	    tzsecs = 86400;
+	} else {
+	    sec += tzsecs;
+	}
+	if (zneg) {
+	    tzsecs = -tzsecs;
+	}
+	if (0 == tzsecs) {
+	    *b-- = '0';
+	} else {
+	    for (; 0 < tzsecs; b--, tzsecs /= 10) {
+		*b = '0' + (tzsecs % 10);
+	    }
+	    if (zneg) {
+		*b-- = '-';
+	    }
+	}
+	*b-- = 'e';
+    }
     if (0 > sec) {
 	neg = 1;
 	sec = -sec;
@@ -1024,7 +1049,7 @@ dump_time(VALUE obj, Out out) {
 	    sec--;
 	}
     }
-    *b-- = '\0';
+    dot = b - 9;
     if (0 < out->opts->sec_prec) {
 	if (9 > out->opts->sec_prec) {
 	    int	i;
@@ -1088,7 +1113,7 @@ dump_xml_time(VALUE obj, Out out) {
     long		nsec = NUM2LONG(rb_funcall2(obj, oj_tv_usec_id, 0, 0)) * 1000;
 #endif
 #endif
-    long		tz_secs = NUM2LONG(rb_funcall2(obj, oj_utc_offset_id, 0, 0));
+    long		tzsecs = NUM2LONG(rb_funcall2(obj, oj_utc_offset_id, 0, 0));
     int			tzhour, tzmin;
     char		tzsign = '+';
 
@@ -1109,16 +1134,16 @@ dump_xml_time(VALUE obj, Out out) {
     }
     // 2012-01-05T23:58:07.123456000+09:00
     //tm = localtime(&sec);
-    sec += tz_secs;
+    sec += tzsecs;
     tm = gmtime(&sec);
 #if 1
-    if (0 > tz_secs) {
+    if (0 > tzsecs) {
         tzsign = '-';
-        tzhour = (int)(tz_secs / -3600);
-        tzmin = (int)(tz_secs / -60) - (tzhour * 60);
+        tzhour = (int)(tzsecs / -3600);
+        tzmin = (int)(tzsecs / -60) - (tzhour * 60);
     } else {
-        tzhour = (int)(tz_secs / 3600);
-        tzmin = (int)(tz_secs / 60) - (tzhour * 60);
+        tzhour = (int)(tzsecs / 3600);
+        tzmin = (int)(tzsecs / 60) - (tzhour * 60);
     }
 #else
     if (0 > tm->tm_gmtoff) {
@@ -1131,7 +1156,7 @@ dump_xml_time(VALUE obj, Out out) {
     }
 #endif
     if (0 == nsec || 0 == out->opts->sec_prec) {
-	if (0 == tzhour && 0 == tzmin) {
+	if (0 == tzsecs && Qtrue == rb_funcall2(obj, oj_utcq_id, 0, 0)) {
 	    sprintf(buf, "%04d-%02d-%02dT%02d:%02d:%02dZ",
 		    tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
 		    tm->tm_hour, tm->tm_min, tm->tm_sec);
@@ -1143,6 +1168,18 @@ dump_xml_time(VALUE obj, Out out) {
 		    tzsign, tzhour, tzmin);
 	    dump_cstr(buf, 25, 0, 0, out);
 	}
+    } else if (0 == tzsecs && Qtrue == rb_funcall2(obj, oj_utcq_id, 0, 0)) {
+	char	format[64] = "%04d-%02d-%02dT%02d:%02d:%02d.%09ldZ";
+	int	len = 30;
+
+	if (9 > out->opts->sec_prec) {
+	    format[32] = '0' + out->opts->sec_prec;
+	    len -= 9 - out->opts->sec_prec;
+	}
+	sprintf(buf, format,
+		tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
+		tm->tm_hour, tm->tm_min, tm->tm_sec, nsec);
+	dump_cstr(buf, len, 0, 0, out);
     } else {
 	char	format[64] = "%04d-%02d-%02dT%02d:%02d:%02d.%09ld%c%02d:%02d";
 	int	len = 35;
@@ -1232,8 +1269,9 @@ dump_data_comp(VALUE obj, int depth, Out out) {
 	    switch (out->opts->time_format) {
 	    case RubyTime:	dump_ruby_time(obj, out);	break;
 	    case XmlTime:	dump_xml_time(obj, out);	break;
+	    case UnixZTime:	dump_time(obj, out, 1);		break;
 	    case UnixTime:
-	    default:		dump_time(obj, out);		break;
+	    default:		dump_time(obj, out, 0);		break;
 	    }
 	} else if (oj_bigdecimal_class == clas) {
 	    volatile VALUE	rstr = rb_funcall(obj, oj_to_s_id, 0);
@@ -1266,9 +1304,12 @@ dump_data_obj(VALUE obj, int depth, Out out) {
 	case XmlTime:
 	    dump_xml_time(obj, out);
 	    break;
+	case UnixZTime:
+	    dump_time(obj, out, 1);
+	    break;
 	case UnixTime:
 	default:
-	    dump_time(obj, out);
+	    dump_time(obj, out, 0);
 	    break;
 	}
 	*out->cur++ = '}';

data/ext/oj/object.c

@@ -29,6 +29,7 @@
  */
 
 #include <stdio.h>
+#include <time.h>
 
 #include "oj.h"
 #include "err.h"
@@ -93,6 +94,146 @@ str_to_value(ParseInfo pi, const char *str, size_t len, const char *orig) {
     return rstr;
 }
 
+#if (RUBY_VERSION_MAJOR == 1 && RUBY_VERSION_MINOR == 8)
+static VALUE
+parse_xml_time(const char *str, int len) {
+    return rb_funcall(rb_cTime, oj_parse_id, 1, rb_str_new(str, len));
+}
+#else
+// The much faster approach (4x faster)
+static int
+parse_num(const char *str, const char *end, int cnt) {
+    int		n = 0;
+    char	c;
+    int		i;
+
+    for (i = cnt; 0 < i; i--, str++) {
+	c = *str;
+	if (end <= str || c < '0' || '9' < c) {
+	    return -1;
+	}
+	n = n * 10 + (c - '0');
+    }
+    return n;
+}
+
+static VALUE
+parse_xml_time(const char *str, int len) {
+    VALUE	args[8];
+    const char	*end = str + len;
+    int		n;
+
+    // year
+    if (0 > (n = parse_num(str, end, 4))) {
+	return Qnil;
+    }
+    str += 4;
+    args[0] = LONG2NUM(n);
+    if ('-' != *str++) {
+	return Qnil;
+    }
+    // month
+    if (0 > (n = parse_num(str, end, 2))) {
+	return Qnil;
+    }
+    str += 2;
+    args[1] = LONG2NUM(n);
+    if ('-' != *str++) {
+	return Qnil;
+    }
+    // day
+    if (0 > (n = parse_num(str, end, 2))) {
+	return Qnil;
+    }
+    str += 2;
+    args[2] = LONG2NUM(n);
+    if ('T' != *str++) {
+	return Qnil;
+    }
+    // hour
+    if (0 > (n = parse_num(str, end, 2))) {
+	return Qnil;
+    }
+    str += 2;
+    args[3] = LONG2NUM(n);
+    if (':' != *str++) {
+	return Qnil;
+    }
+    // minute
+    if (0 > (n = parse_num(str, end, 2))) {
+	return Qnil;
+    }
+    str += 2;
+    args[4] = LONG2NUM(n);
+    if (':' != *str++) {
+	return Qnil;
+    }
+    // second
+    if (0 > (n = parse_num(str, end, 2))) {
+	return Qnil;
+    }
+    str += 2;
+    if (str == end) {
+	args[5] = LONG2NUM(n);
+	args[6] = LONG2NUM(0);
+    } else {
+	char	c = *str++;
+
+	if ('.' == c) {
+	    long	nsec = 0;
+
+	    for (; str < end; str++) {
+		c = *str;
+		if (c < '0' || '9' < c) {
+		    str++;
+		    break;
+		}
+		nsec = nsec * 10 + (c - '0');
+	    }
+	    args[5] = rb_float_new((double)n + ((double)nsec + 0.5) / 1000000000.0);
+	} else {
+	    args[5] = LONG2NUM(n);
+	}
+	if (end < str) {
+	    args[6] = LONG2NUM(0);
+	} else {
+	    if ('Z' == c) {
+		return rb_funcall2(rb_cTime, oj_utc_id, 6, args);
+	    } else if ('+' == c) {
+		int	hr = parse_num(str, end, 2);
+		int	min;
+
+		str += 2;
+		if (0 > hr || ':' != *str++) {
+		    return Qnil;
+		}
+		min = parse_num(str, end, 2);
+		if (0 > min) {
+		    return Qnil;
+		}
+		args[6] = LONG2NUM(hr * 3600 + min * 60);
+	    } else if ('-' == c) {
+		int	hr = parse_num(str, end, 2);
+		int	min;
+
+		str += 2;
+		if (0 > hr || ':' != *str++) {
+		    return Qnil;
+		}
+		min = parse_num(str, end, 2);
+		if (0 > min) {
+		    return Qnil;
+		}
+		args[6] = LONG2NUM(-(hr * 3600 + min * 60));
+	    } else {
+		args[6] = LONG2NUM(0);
+	    }
+	}
+    }
+    return rb_funcall2(rb_cTime, oj_new_id, 7, args);
+}
+#endif
+
 static int
 hat_cstr(ParseInfo pi, Val parent, Val kval, const char *str, size_t len) {
     const char	*key = kval->key;
@@ -133,7 +274,7 @@ hat_cstr(ParseInfo pi, Val parent, Val kval, const char *str, size_t len) {
 	    parent->val = oj_name2class(pi, str, len, Yes == pi->options.auto_define);
 	    break;
 	case 't': // time
-	    parent->val = rb_funcall(oj_time_class, rb_intern("parse"), 1, rb_str_new(str, len));
+	    parent->val = parse_xml_time(str, len);
 	    break;
 	default:
 	    return 0;
@@ -159,11 +300,55 @@ hat_num(ParseInfo pi, Val parent, Val kval, NumInfo ni) {
 			nsec = 1000000000LL - nsec;
 		    }
 		}
+		if (86400 == ni->exp) { // UTC time
+#if HAS_NANO_TIME
+		    parent->val = rb_time_nano_new(ni->i, (long)nsec);
+#else
+		    parent->val = rb_time_new(ni->i, (long)(nsec / 1000));
+#endif
+		    // Since the ruby C routines alway create local time, the
+		    // offset and then a convertion to UTC keeps makes the time
+		    // match the expected value.
+		    parent->val = rb_funcall2(parent->val, oj_utc_id, 0, 0);
+		} else if (ni->hasExp) {
+		    time_t	t = (time_t)ni->i;
+		    struct tm	*st = gmtime(&t);
+#if RUBY_VERSION_MAJOR == 1 && RUBY_VERSION_MINOR == 8
+		    // The only methods that allow the UTC offset to be set in
+		    // 1.8.7 is the parse() and xmlschema() methods. The
+		    // xmlschema() method always returns a Time instance that is
+		    // UTC time. (true on some platforms anyway) Time.parse()
+		    // fails on other Ruby versions until 2.2.0.
+		    char	buf[64];
+		    int		z = (0 > ni->exp ? -ni->exp : ni->exp) / 60;
+		    int		tzhour = z / 60;
+		    int		tzmin = z - tzhour * 60;
+		    int		cnt;
+
+		    cnt = sprintf(buf, "%04d-%02d-%02dT%02d:%02d:%02d.%09ld%c%02d:%02d",
+				  1900 + st->tm_year, 1 + st->tm_mon, st->tm_mday,
+				  st->tm_hour, st->tm_min, st->tm_sec, (long)nsec,
+				  (0 > ni->exp ? '-' : '+'), tzhour, tzmin);
+		    parent->val = rb_funcall(rb_cTime, oj_parse_id, 1, rb_str_new(buf, cnt));
+#else
+		    VALUE	args[8];
+
+		    args[0] = LONG2NUM(1900 + st->tm_year);
+		    args[1] = LONG2NUM(1 + st->tm_mon);
+		    args[2] = LONG2NUM(st->tm_mday);
+		    args[3] = LONG2NUM(st->tm_hour);
+		    args[4] = LONG2NUM(st->tm_min);
+		    args[5] = rb_float_new((double)st->tm_sec + ((double)nsec + 0.5) / 1000000000.0);
+		    args[6] = LONG2NUM(ni->exp);
+		    parent->val = rb_funcall2(rb_cTime, oj_new_id, 7, args);
+#endif
+		} else {
 #if HAS_NANO_TIME
-		parent->val = rb_time_nano_new(ni->i, (long)nsec);
+		    parent->val = rb_time_nano_new(ni->i, (long)nsec);
 #else
-		parent->val = rb_time_new(ni->i, (long)(nsec / 1000));
+		    parent->val = rb_time_new(ni->i, (long)(nsec / 1000));
 #endif
+		}
 	    }
 	    break;
 	case 'i': // circular index

data/ext/oj/oj.c

@@ -70,6 +70,7 @@ ID	oj_instance_variables_id;
 ID	oj_json_create_id;
 ID	oj_length_id;
 ID	oj_new_id;
+ID	oj_parse_id;
 ID	oj_pos_id;
 ID	oj_read_id;
 ID	oj_readpartial_id;
@@ -84,7 +85,9 @@ ID	oj_to_time_id;
 ID	oj_tv_nsec_id;
 ID	oj_tv_sec_id;
 ID	oj_tv_usec_id;
+ID	oj_utc_id;
 ID	oj_utc_offset_id;
+ID	oj_utcq_id;
 ID	oj_write_id;
 
 VALUE	oj_bag_class;
@@ -97,7 +100,6 @@ VALUE	oj_stream_writer_class;
 VALUE	oj_string_writer_class;
 VALUE	oj_stringio_class;
 VALUE	oj_struct_class;
-VALUE	oj_time_class;
 
 VALUE	oj_slash_string;
 
@@ -130,6 +132,7 @@ static VALUE	strict_sym;
 static VALUE	symbol_keys_sym;
 static VALUE	time_format_sym;
 static VALUE	unix_sym;
+static VALUE	unix_zone_sym;
 static VALUE	use_to_json_sym;
 static VALUE	xmlschema_sym;
 static VALUE	xss_safe_sym;
@@ -164,7 +167,7 @@ struct _Options	oj_default_options = {
     JSONEsc,		// escape_mode
     ObjectMode,		// mode
     Yes,		// class_cache
-    UnixTime,		// time_format
+    UnixZTime,		// time_format
     Yes,		// bigdec_as_num
     AutoDec,		// bigdec_load
     Yes,		// to_json
@@ -191,7 +194,7 @@ static VALUE	define_mimic_json(int argc, VALUE *argv, VALUE self);
  * - escape_mode: [:newline|:json|:xss_safe|:ascii|nil] determines the characters to escape
  * - class_cache: [true|false|nil] cache classes for faster parsing (if dynamically modifying classes or reloading classes then don't use this)
  * - mode: [:object|:strict|:compat|:null] load and dump modes to use for JSON
- * - time_format: [:unix|:xmlschema|:ruby] time format when dumping in :compat mode
+ * - time_format: [:unix|:unix_zone|:xmlschema|:ruby] time format when dumping in :compat and :object mode
  * - bigdecimal_as_decimal: [true|false|nil] dump BigDecimal as a decimal number or as a String
  * - bigdecimal_load: [:bigdecimal|:float|:auto] load decimals as BigDecimal instead of as a Float. :auto pick the most precise for the number of digits.
  * - create_id: [String|nil] create id for json compatible object encoding, default is 'json_create'
@@ -236,6 +239,7 @@ get_def_opts(VALUE self) {
     switch (oj_default_options.time_format) {
     case XmlTime:	rb_hash_aset(opts, time_format_sym, xmlschema_sym);	break;
     case RubyTime:	rb_hash_aset(opts, time_format_sym, ruby_sym);		break;
+    case UnixZTime:	rb_hash_aset(opts, time_format_sym, unix_zone_sym);	break;
     case UnixTime:
     default:		rb_hash_aset(opts, time_format_sym, unix_sym);		break;
     }
@@ -275,6 +279,7 @@ get_def_opts(VALUE self) {
  *	  replaces them with a null.
  * @param [:unix|:xmlschema|:ruby] time format when dumping in :compat mode
  *        :unix decimal number denoting the number of seconds since 1/1/1970,
+ *        :unix_zone decimal number denoting the number of seconds since 1/1/1970 plus the utc_offset in the exponent ,
  *        :xmlschema date-time format taken from XML Schema as a String,
  *        :ruby Time.to_s formatted String
  * @param [String|nil] :create_id create id for json compatible object encoding
@@ -360,12 +365,14 @@ set_def_opts(VALUE self, VALUE opts) {
 	// ignore
     } else if (unix_sym == v) {
 	oj_default_options.time_format = UnixTime;
+    } else if (unix_zone_sym == v) {
+	oj_default_options.time_format = UnixZTime;
     } else if (xmlschema_sym == v) {
 	oj_default_options.time_format = XmlTime;
     } else if (ruby_sym == v) {
 	oj_default_options.time_format = RubyTime;
     } else {
-	rb_raise(rb_eArgError, ":time_format must be :unix, :xmlschema, or :ruby.");
+	rb_raise(rb_eArgError, ":time_format must be :unix, :unix_zone, :xmlschema, or :ruby.");
     }
 
     v = rb_hash_lookup(opts, escape_mode_sym);
@@ -512,12 +519,14 @@ oj_parse_options(VALUE ropts, Options copts) {
 	if (Qnil != (v = rb_hash_lookup(ropts, time_format_sym))) {
 	    if (unix_sym == v) {
 		copts->time_format = UnixTime;
+	    } else if (unix_zone_sym == v) {
+		copts->time_format = UnixZTime;
 	    } else if (xmlschema_sym == v) {
 		copts->time_format = XmlTime;
 	    } else if (ruby_sym == v) {
 		copts->time_format = RubyTime;
 	    } else {
-		rb_raise(rb_eArgError, ":time_format must be :unix, :xmlschema, or :ruby.");
+		rb_raise(rb_eArgError, ":time_format must be :unix, :unix_zone, :xmlschema, or :ruby.");
 	    }
 	}
 
@@ -2042,6 +2051,7 @@ void Init_oj() {
     oj_json_create_id = rb_intern("json_create");
     oj_length_id = rb_intern("length");
     oj_new_id = rb_intern("new");
+    oj_parse_id = rb_intern("parse");
     oj_pos_id = rb_intern("pos");
     oj_read_id = rb_intern("read");
     oj_readpartial_id = rb_intern("readpartial");
@@ -2056,7 +2066,9 @@ void Init_oj() {
     oj_tv_nsec_id = rb_intern("tv_nsec");
     oj_tv_sec_id = rb_intern("tv_sec");
     oj_tv_usec_id = rb_intern("tv_usec");
+    oj_utc_id = rb_intern("utc");
     oj_utc_offset_id = rb_intern("utc_offset");
+    oj_utcq_id = rb_intern("utc?");
     oj_write_id = rb_intern("write");
 
     rb_require("oj/bag");
@@ -2072,7 +2084,6 @@ void Init_oj() {
     oj_parse_error_class = rb_const_get_at(Oj, rb_intern("ParseError"));
     oj_stringio_class = rb_const_get(rb_cObject, rb_intern("StringIO"));
     oj_struct_class = rb_const_get(rb_cObject, rb_intern("Struct"));
-    oj_time_class = rb_const_get(rb_cObject, rb_intern("Time"));
 
     allow_gc_sym = ID2SYM(rb_intern("allow_gc"));	rb_gc_register_address(&allow_gc_sym);
     ascii_only_sym = ID2SYM(rb_intern("ascii_only"));	rb_gc_register_address(&ascii_only_sym);
@@ -2103,6 +2114,7 @@ void Init_oj() {
     symbol_keys_sym = ID2SYM(rb_intern("symbol_keys"));	rb_gc_register_address(&symbol_keys_sym);
     time_format_sym = ID2SYM(rb_intern("time_format"));	rb_gc_register_address(&time_format_sym);
     unix_sym = ID2SYM(rb_intern("unix"));		rb_gc_register_address(&unix_sym);
+    unix_zone_sym = ID2SYM(rb_intern("unix_zone"));	rb_gc_register_address(&unix_zone_sym);
     use_to_json_sym = ID2SYM(rb_intern("use_to_json"));	rb_gc_register_address(&use_to_json_sym);
     xmlschema_sym = ID2SYM(rb_intern("xmlschema"));	rb_gc_register_address(&xmlschema_sym);
     xss_safe_sym = ID2SYM(rb_intern("xss_safe"));	rb_gc_register_address(&xss_safe_sym);