Addressed danno's comments

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@15823 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/deoptimizer.cc b/src/deoptimizer.cc
index 5a362ce..72cf3c0 100644 (file)
--- a/src/deoptimizer.cc
+++ b/src/deoptimizer.cc
@@ -1265,8 +1265,8 @@ void Deoptimizer::DoComputeAccessorStubFrame(TranslationIterator* iterator,
   // MacroAssembler::EnterFrame). For a setter stub frame we need one additional
   // entry for the implicit return value, see
   // StoreStubCompiler::CompileStoreViaSetter.
-  unsigned fixed_frame_entries = (kPCOnStackSize/kPointerSize) +
-                                 (kFPOnStackSize/kPointerSize) + 3 +
+  unsigned fixed_frame_entries = (kPCOnStackSize / kPointerSize) +
+                                 (kFPOnStackSize / kPointerSize) + 3 +
                                  (is_setter_stub_frame ? 1 : 0);
   unsigned fixed_frame_size = fixed_frame_entries * kPointerSize;
   unsigned output_frame_size = height_in_bytes + fixed_frame_size;

diff --git a/src/deoptimizer.h b/src/deoptimizer.h
index eb9cc7b..b29c1cb 100644 (file)
--- a/src/deoptimizer.h
+++ b/src/deoptimizer.h
@@ -473,7 +473,6 @@ class FrameDescription {
  public:
   FrameDescription(uint32_t frame_size,
                    JSFunction* function);
-  virtual ~FrameDescription() { }
 
   void* operator new(size_t size, uint32_t frame_size) {
     // Subtracts kPointerSize, as the member frame_content_ already supplies
@@ -511,9 +510,9 @@ class FrameDescription {
     *GetFrameSlotPointer(offset) = value;
   }
 
-  virtual void SetCallerPc(unsigned offset, intptr_t value);
+  void SetCallerPc(unsigned offset, intptr_t value);
 
-  virtual void SetCallerFp(unsigned offset, intptr_t value);
+  void SetCallerFp(unsigned offset, intptr_t value);
 
   intptr_t GetRegister(unsigned n) const {
     ASSERT(n < ARRAY_SIZE(registers_));