gvars 0.9.11 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dd8a8d29639c250dbe4ef1354ca98ce1e94510753c111ba62965af0f8bca77cb
-  data.tar.gz: f1d88f86c4fba91ad7e842a17c00b1ec8ee1038bb7780ff6a4368eadcc133443
+  metadata.gz: 6cec95dcb820444abd65614b00b8d58228f257aa801c57fd258d205cdfa3cca6
+  data.tar.gz: '0343590213ebc76ede529e8e469bebd69439c62826ef134852d37870da417f16'
 SHA512:
-  metadata.gz: ccea699c50675a7f3a6d65d52b684d6acb77ff99177c019111f48fdf724cd4c3753bdb70e65b02437ddf259187104472f9bbb149370d2881a7c9bbb6912b2f72
-  data.tar.gz: 8e2344c225d5f304391bfdadf51f9f416d41a1c0bcdd83755fcc11d9b20b1ed3851e4c822aee286de9b4022594e06666e45a93e324ec608e8be96908ef2c4cc0
+  metadata.gz: 0db69cfd69b689e2d93f5b62780c197b28929933b43d77241e89bd1658f52d676e637d1f1943a27723e8a9ba5179b3ba0bb8c9c229b768ff82e92c9431dcd560
+  data.tar.gz: 1ecd051d8c68d123131c2ef8d96917c70cb17fe6f08063c9525ba0afdca29c6dd053eb0dd3aad2a35af151c919d61d27e7fc7ba54eab7dc58660ad3b441a7790

data/README.md

@@ -31,8 +31,8 @@ puts $TIME.monday? #=> true
 
 # You can also provide a custom setter
 GVars.virtual(:$PWD,
-	getter: ->{ Dir.pwd },
-	setter: ->new{ Dir.chdir(new) })
+  getter: ->{ Dir.pwd },
+  setter: ->new{ Dir.chdir(new) })
 $PWD = '/tmp'
 p $PWD #=> "/private/tmp" (macOS symlinks `/tmp` to `/private/tmp`)
 ```
@@ -51,9 +51,9 @@ p $COUNTER #=> 11
 # You can also specify a state, which is given to the getter (and setter),
 # but isn't updated by the getter's return value.
 GVars.hooked(:$RAND,
-	state: Random.new,
-	getter: ->random{ random.rand },
-	setter: ->(value,random){ random.srand(value) }) # doesn't actually work.. `Random#srand` isn't a thing
+  state: Random.new,
+  getter: ->random{ random.rand },
+  setter: ->(value,random){ random.srand(value) }) # doesn't actually work.. `Random#srand` isn't a thing
 ```
 
 ## `OptParse` support
@@ -64,19 +64,19 @@ require 'optparse'
 require 'gvars'
 
 OptParse.new do |op|
-	op.on '--[no-]-read'
-	op.on '--[no-]-write'
-	op.on '--[no-]-execute'
-	op.on '--timeout=SECONDS', Float
-	op.on '--cache-dir=PATH'
-
-	op.parse!(
-		%w[--read --no-execute --timeout=10 --cache-dir=/foo/bar],
-		into: GVars
-	)
-
-	p [$read, $write, $execute, $timeout, $cache_dir]
-	#=> [true, nil, false, 10.0, "/foo/bar"]
+  op.on '--[no-]read'
+  op.on '--[no-]write'
+  op.on '--[no-]execute'
+  op.on '--timeout=SECONDS', Float
+  op.on '--cache-dir=PATH'
+
+  op.parse!(
+    %w[--read --no-execute --timeout=10 --cache-dir=/foo/bar],
+    into: GVars
+  )
+
+  p [$read, $write, $execute, $timeout, $cache_dir]
+  #=> [true, nil, false, 10.0, "/foo/bar"]
 end
 ```
 

data/ext/gvars/gvars.c

@@ -2,6 +2,20 @@
 #define dbg(...) (fprintf(stderr, "%s:%s:%d: ", __FILE__, __FUNCTION__, __LINE__), fprintf(stderr,__VA_ARGS__),fprintf(stderr,"\n"),fflush(stderr))
 
 VALUE gvars_module;
+static ID id_debug;
+
+static bool
+gvars_debug_enabled(void)
+{
+	VALUE debug_mode = rb_ivar_get(gvars_module, id_debug);
+	return debug_mode == Qnil ? ruby_debug : RTEST(debug_mode);
+}
+
+static VALUE
+gvars_f_defined_p(VALUE self)
+{
+	return gvars_debug_enabled() ? Qtrue : Qfalse;
+}
 
 // Converts `name` to a global variable name, ensures it's valid, and returns it.
 //
@@ -29,8 +43,6 @@ static char *get_global_name(VALUE *name, bool translate) {
 	return namestr;
 }
 
-
-
 static VALUE
 gvars_f_global_variable_get(VALUE self, VALUE name)
 {
@@ -49,11 +61,23 @@ gvars_f_global_variable_set(VALUE self, VALUE name, VALUE value)
 	return rb_gv_set(get_global_name(&name, false), value);
 }
 
-
 static VALUE
 gvars_f_global_variable_aset(VALUE self, VALUE name, VALUE value)
 {
-	return rb_gv_set(get_global_name(&name, true), value);
+	char *sname = get_global_name(&name, true);
+	if (gvars_debug_enabled()) {
+		char *sname_without_dollar = sname;
+		if (sname_without_dollar[0] == '$') {
+			++sname_without_dollar;
+		}
+
+		VALUE msg = rb_sprintf("GVars: setting global $%s to: %"PRIsVALUE"\n",
+			sname_without_dollar, rb_inspect(value));
+
+		rb_io_write(rb_gv_get("$stderr"), msg);
+	}
+
+	return rb_gv_set(sname, value);
 }
 
 static VALUE
@@ -359,6 +383,7 @@ void
 Init_gvars(void)
 {
 	gvars_module = rb_define_module("GVars");
+	VALUE gvars_singleton = rb_singleton_class(gvars_module);
 	rb_ivar_set(gvars_module, rb_intern("vars"), rb_hash_new());
 
     virtual_method_keyword_ids[VIRTUAL_METHOD_KEYWORDS_GETTER] = rb_intern("getter");
@@ -367,6 +392,12 @@ Init_gvars(void)
     virtual_method_keyword_ids[VIRTUAL_METHOD_KEYWORDS_STATE] = rb_intern("state");
     virtual_method_keyword_ids[VIRTUAL_METHOD_KEYWORDS_READONLY] = rb_intern("readonly");
 
+
+    // Define debugging construct
+    id_debug = rb_intern_const("@debug");
+    rb_define_attr(gvars_singleton, "debug", 1, 1);
+    rb_define_singleton_method(gvars_module, "debug?", gvars_f_defined_p, 0);
+
 	// Define module-level functions that can be used as mixins
 	rb_define_module_function(gvars_module, "global_variable_get", gvars_f_global_variable_get, 1);
 	rb_define_module_function(gvars_module, "global_variable_set", gvars_f_global_variable_set, 2);
@@ -388,7 +419,6 @@ Init_gvars(void)
 	rb_define_singleton_method(gvars_module, "[]=", gvars_f_global_variable_aset, 2);
 
 	// Aliases
-	VALUE gvars_singleton = rb_singleton_class(gvars_module);
 	rb_define_alias(gvars_singleton, "get", "global_variable_get");
 	rb_define_alias(gvars_singleton, "set", "global_variable_set");
 	rb_define_alias(gvars_singleton, "alias", "alias_global_variable");

data/lib/gvars/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GVars
-  VERSION = '0.9.11'
+  VERSION = '0.10.1'
 end

data/sig/gvars.rbs

@@ -1,6 +1,9 @@
 module GVars
   VERSION: String
 
+  attr_accessor self.debug: bool?
+  def self.debug?: () -> bool
+
   # Mixin methods
   def self?.global_variable_get: (interned name) -> untyped
   def self?.global_variable_set: [T] (interned name, T value) -> T

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gvars
 version: !ruby/object:Gem::Version
-  version: 0.9.11
+  version: 0.10.1
 platform: ruby
 authors:
 - Sam Westerman