Skip to content

Conversation

@sampersand
Copy link
Contributor

@sampersand sampersand commented Oct 17, 2025

This PR adds Kernel.trace_var and Kernel.untrace_var in, along with the Kernel::_Tracer interface.

This is a pretty simple PR to ease back into things!

@sampersand sampersand marked this pull request as ready for review October 17, 2025 20:11
@sampersand sampersand force-pushed the sampersand/2025-10-13/add-tracevar branch from 6535a83 to c9b056a Compare October 17, 2025 20:14
@ksss
Copy link
Collaborator

ksss commented Oct 24, 2025

I agree with adding the methods, but I'm unsure how much management cost we should invest in debugging methods.

@sampersand
Copy link
Contributor Author

I don't see a problem with adding them in. I've also used them in code before, such as:

class Logger
  class << self; attr_accessor :debug_lvl end
  trace_var :$DEBUG, &method(:debug_lvl=)
  trace_var :$-d, &method(:debug_lvl=)
end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants