optparse2 0.7.6 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '089aa30260171fcc53efa5d5099338973990015878df07b4a3c9ae023ae7d0c6'
-  data.tar.gz: 2e46af429af2a2c3576bb5bfd2f32bc4bcb448fb2ecb73c34f419de705d502d6
+  metadata.gz: 7af25aed35f6e83c5d21f2243c8cc714f182f32a5f012ad93dd04486e8ba5865
+  data.tar.gz: c7804d256d8ead6a97f9edbcd6049ccb9d9226a297341a8fe6888b39133023c3
 SHA512:
-  metadata.gz: c2f53a2bc1382025cdc0c66ddbfe2878bbcec9a21ddb66b5c54e6952fdef9a32d4b9d9c11ea68875ec2b1e875f135cd33c48ec54d138cc450c7d92428e9a8ed0
-  data.tar.gz: 66b6650a5aa9b38020ccb8a053268037188897b7b6d2514b14b384fb488b1d7553f005bd9ed266b4ae293ad506cc45ad6f8fe6384125bbc9928e1b9b79b1d42b
+  metadata.gz: 00fe7339a1b17afec474b3e1719ea7b774c2ffb5ae55c73310d81c9907a804958909160d7c04df4efbd626ee6a8eb3f85d969b28508f57ed74c0a8090ea2384b
+  data.tar.gz: c01111aa1350878a7a060a3bec5a283465349a0c4bbe498a48d22b59d1c3f15736e9d180b9eed127c02a5c6a67e0e2ff973e3aad31e54e3109203ab7a5a9d155

data/lib/optparse2/fixes.rb

@@ -11,9 +11,4 @@ class OptParse2
   def define_head(*opts, **, &block) base.append(*(sw = make_switch(opts, block, **))); sw[0] end
   alias def_tail_option define_tail
   def on_tail(...) define_tail(...); self end
-
-  def initialize(*)
-    @defaults = {}
-    super
-  end
 end

data/lib/optparse2/switch-helpers.rb

@@ -150,7 +150,7 @@ class OptParse2
     # Default values of switches are used when the switch is never passed in.
     # If the `value` that's provided doesn't respond to `.call`, it's converted to a proc.
     # If `bypass` is truthy, then the default value is never passed to the block's proc (if any)
-    def set_default(value, description, bypass, no_default_description)
+    def set_default(value, description, bypass)
       if required?
         raise ArgumentError, 'cannot supply a default value for a required argument'
       end
@@ -165,7 +165,6 @@ class OptParse2
         @default_proc = proc { |_switch_name| value }
       end
 
-      @no_default_description = no_default_description
       @default_description = description
       @default_bypass = bypass
     end
@@ -173,16 +172,30 @@ class OptParse2
     # Calls the default proc to figure out what the default value is for this switch
     def default_bypass? = @default_bypass
     def default? = defined?(@default_proc)
-    def default = @default_proc&.call(switch_name)
+    def default
+      return unless default?
+      if defined? @__default_value_after_calling_proc__
+        @__default_value_after_calling_proc__
+      else
+        @__default_value_after_calling_proc__ = @default_proc&.call(switch_name)
+      end
+    end
 
-    def default_description = @default_description || default.to_s
     def desc
-      return super unless defined? @default_proc
-      return super if @no_default_description
-      x = super
-      x << '' if x.empty?
-      x[-1] += " [default: #{default_description}]"
-      x
+      super_desc = super
+      return super_desc unless defined? @default_proc
+
+      if !@default_description
+        return super_desc
+      end
+
+      super_desc << '' if super_desc.empty?
+      super_desc[-1] += " [default"
+      if @default_description != true
+        super_desc.last.concat ": #{@default_description == :__TODO_DEFAULT__ ? default : @default_description}"
+      end
+      super_desc.last.concat ']'
+      super_desc
     end
   end
 end

data/lib/optparse2/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class OptParse2
-  VERSION = "0.7.6"
+  VERSION = "0.8.0"
 end

data/lib/optparse2.rb

@@ -35,7 +35,7 @@ class OptParse2
     key: @group,
     default: nodefault=true,
     default_bypass: false,
-    default_description: no_default_description=true,
+    default_description: :__TODO_DEFAULT__,
     required: false,
     multiple: nil
   )
@@ -56,12 +56,12 @@ class OptParse2
 
     sw.set_required true if required
 
-    if nodefault && !no_default_description
+    if nodefault && default_description != :__TODO_DEFAULT__
       raise ArgumentError, "default: not supplied, but default_description: given"
     elsif nodefault && default_bypass
       raise ArgumentError, "default: not supplied, but default_bypass: given"
     elsif not nodefault
-      sw.set_default(default, default_description, default_bypass, !no_default_description)
+      sw.set_default(default, default_description, default_bypass)
       @defaults << sw
     end
 
@@ -270,6 +270,17 @@ class OptParse2
   end
 end
 
+__END__
+
+OptParse2.new do |op|
+  op.switch('-f', '--foo=BAR', 'does it')
+    .default { p $x += 1 }
+    .build!
+  op.parse! into: opts={}
+  p opts
+  p $x
+end
+
 __END__
 OptParse2.new do |op|
   op.on '-f', '--foo=BAR', 'does it', default: 10, default_description: nil

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: optparse2
 version: !ruby/object:Gem::Version
-  version: 0.7.6
+  version: 0.8.0
 platform: ruby
 authors:
 - SamW