Results 589 comments of MSP-Greg

As I said ... > Did you try it with code? In the SU console: ``` 'format %s %s' % %w[this text] => "format this text" 10 % 3 =>...

> **So again I think that the warning itself is UNNECESSARY and therefore a bug.** Well, a quick check shows that the warning has existed since at least Ruby 2.2.4p230....

The more you post, the more apparent it is that you know very little about how Ruby and gems work outside of SketchUp. Ever written, reviewed, or contributed to parser...

Note that `$env:RUBYOPT = '-W2'` and `$env:RUBYOPT = '--verbose'` are equivalent. Duplicate of #300 from Jul 29, 2019.

I just checked this with SU 2025 on Windows. Just two warnings: ``` C:/Program Files/SketchUp/SketchUp 2025/SketchUp/Tools/sketchup.rb:32: warning: global variable `$loaded_files' not initialized C:/Users//AppData/Roaming/SketchUp/SketchUp 2025/SketchUp/Plugins/su_revit_importer_installer.rb:25: warning: assigned but unused variable -...

This looks like a good addition. Looking at https://github.com/rack/rack/issues/1777, https://github.com/rack/rack/pull/1802, https://github.com/rack/rack/pull/1952, and https://github.com/Shopify/pitchfork/pull/98. Not sure if the call loop should be updated to match what the pitchfork code did, I...

`rack.response_finished` should be added to Puma. [Add `rack.after_reply` to simplify logging and resources](https://github.com/puma/puma/commit/be4a8336c0b4fc911b99d1ffddc4733b6f38d81d) was added 20-Oct-2011. It also exists in Unicorn. Since the rack spec does state 'They should be...

> I think that's useful, could you make a PR with that? I will. But, now that you mention 'timeout', some of the debug failures were SEGV or timeouts, and...

```ruby orig_out = $stdout.dup orig_err = $stderr.dup path = ENV['TMPDIR'] || ENV['TEMPDIR'] File.open("#{path}/stdout.log", mode: 'w+') { |f| $stdout.reopen f } File.open("#{path}/stderr.log", mode: 'w+') { |f| $stderr.reopen f } puts 'Hello...