Permalink
Browse files

Remove unused -Xthread.debug option

Debugging information for threads is set at compile time
not runtime. This patch removes the unused -Xthread.debug option
along with two redundant messages that were already controlled
by the cDebugThreading variable set at compile time.
  • Loading branch information...
1 parent eb4a923 commit 50166cec2f4882df405fb378b85838ab19d1e9cb @southerngs southerngs committed Feb 6, 2014
Showing with 0 additions and 15 deletions.
  1. +0 −3 library/rubinius/configuration.rb
  2. +0 −12 vm/oop.cpp
@@ -181,8 +181,5 @@
c.vm_variable "vm.crash_report_path", :string,
:as => "report_path",
:description => "Set a custom path to write crash reports"
-
- c.vm_variable "thread.debug", false,
- "Print threading notices when they occur"
end
View
@@ -578,11 +578,6 @@ namespace rubinius {
case eAuxWordEmpty:
case eAuxWordObjID:
case eAuxWordHandle:
- // Um. well geez. We don't have this object locked.
- if(state->shared().config.thread_debug) {
- std::cerr << "[THREAD] Attempted to unlock an unlocked object.\n";
- }
-
if(cDebugThreading) {
std::cerr << "[LOCK " << state->vm()->thread_id() << " attempted to unlock an unlocked header]\n";
}
@@ -598,13 +593,6 @@ namespace rubinius {
<< "]\n";
}
- if(state->shared().config.thread_debug) {
- std::cerr
- << "[THREAD] Attempted to unlock an object locked by other thread."
- << "locker=" << locker_tid
- << ", current=" << state->vm()->thread_id()
- << "\n";
- }
return eLockError;
}

0 comments on commit 50166ce

Please sign in to comment.