Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hash with long-computable inspect fix #111

Merged
merged 5 commits into from
Aug 7, 2017
Merged

Conversation

ViugiNick
Copy link
Collaborator

format xml_printer with RM formatter and execution time control for calculating hash keys and values

Copy link
Contributor

@valich valich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix typos and rebase on current master

@@ -159,11 +165,10 @@ def print_string(string)
end
end

def exec_with_allocation_control(value, memory_limit, time_limit, exec_method, return_message_if_overflow)
def exec_with_allocation_control(value, memory_limit, time_limit, exec_method, overflow_massage_type)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

massage?

@@ -5,6 +5,12 @@

module Debugger

module OverflowMessageType
NIL_MESSAGE = lambda {|e| nil}
EXEPTION_MESSAGE = lambda {|e| e.message}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exception

@ViugiNick ViugiNick force-pushed the hash-fix branch 3 times, most recently from 52dd2f4 to cbdb160 Compare August 3, 2017 12:12
@valich valich merged commit 828f0d8 into ruby-debug:master Aug 7, 2017
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