diff options
Diffstat (limited to 'src/processor')
-rw-r--r-- | src/processor/stack_frame_symbolizer.cc | 24 | ||||
-rw-r--r-- | src/processor/stackwalker.cc | 4 |
2 files changed, 14 insertions, 14 deletions
diff --git a/src/processor/stack_frame_symbolizer.cc b/src/processor/stack_frame_symbolizer.cc index 03e14504..8e020c98 100644 --- a/src/processor/stack_frame_symbolizer.cc +++ b/src/processor/stack_frame_symbolizer.cc @@ -59,27 +59,27 @@ StackFrameSymbolizer::SymbolizerResult StackFrameSymbolizer::FillSourceLineInfo( StackFrame* frame) { assert(frame); - if (!modules) return ERROR; + if (!modules) return kError; const CodeModule* module = modules->GetModuleForAddress(frame->instruction); - if (!module) return ERROR; + if (!module) return kError; frame->module = module; - if (!resolver_) return ERROR; // no resolver. + if (!resolver_) return kError; // no resolver. // If module is known to have missing symbol file, return. if (no_symbol_modules_.find(module->code_file()) != no_symbol_modules_.end()) { - return ERROR; + return kError; } // If module is already loaded, go ahead to fill source line info and return. if (resolver_->HasModule(frame->module)) { resolver_->FillSourceLineInfo(frame); - return NO_ERROR; + return kNoError; } // Module needs to fetch symbol file. First check to see if supplier exists. if (!supplier_) { - return ERROR; + return kError; } // Start fetching symbol from supplier. @@ -98,26 +98,26 @@ StackFrameSymbolizer::SymbolizerResult StackFrameSymbolizer::FillSourceLineInfo( if (load_success) { resolver_->FillSourceLineInfo(frame); - return NO_ERROR; + return kNoError; } else { BPLOG(ERROR) << "Failed to load symbol file in resolver."; no_symbol_modules_.insert(module->code_file()); - return ERROR; + return kError; } } case SymbolSupplier::NOT_FOUND: no_symbol_modules_.insert(module->code_file()); - return ERROR; + return kError; case SymbolSupplier::INTERRUPT: - return INTERRUPT; + return kInterrupt; default: BPLOG(ERROR) << "Unknown SymbolResult enum: " << symbol_result; - return ERROR; + return kError; } - return ERROR; + return kError; } WindowsFrameInfo* StackFrameSymbolizer::FindWindowsFrameInfo( diff --git a/src/processor/stackwalker.cc b/src/processor/stackwalker.cc index 37664d41..58289425 100644 --- a/src/processor/stackwalker.cc +++ b/src/processor/stackwalker.cc @@ -90,7 +90,7 @@ bool Stackwalker::Walk(CallStack* stack) { StackFrameSymbolizer::SymbolizerResult symbolizer_result = frame_symbolizer_->FillSourceLineInfo(modules_, system_info_, frame.get()); - if (symbolizer_result == StackFrameSymbolizer::INTERRUPT) { + if (symbolizer_result == StackFrameSymbolizer::kInterrupt) { BPLOG(INFO) << "Stack walk is interrupted."; return false; } @@ -185,7 +185,7 @@ bool Stackwalker::InstructionAddressSeemsValid(u_int64_t address) { return true; } - if (symbolizer_result != StackFrameSymbolizer::NO_ERROR) { + if (symbolizer_result != StackFrameSymbolizer::kNoError) { // Some error occurred during symbolization, but the address is within a // known module return true; |