Reviewers: Sven Panne,

Message:
Trivial one... PTAL

Description:
TypePrint() does not depend on Object::Print().

Please review this at https://codereview.chromium.org/225353004/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+0, -6 lines):
  M src/types.h
  M src/types.cc


Index: src/types.cc
diff --git a/src/types.cc b/src/types.cc
index 804ea5ece64d8a12941040014fb2300d5e59c2bd..9cab090695037aff9768a9a6c2de746ddbf050aa 100644
--- a/src/types.cc
+++ b/src/types.cc
@@ -568,7 +568,6 @@ Representation Representation::FromType(Type* type) {
 }


-#ifdef OBJECT_PRINT
 template<class Config>
 void TypeImpl<Config>::TypePrint(PrintDimension dim) {
   TypePrint(stdout, dim);
@@ -666,7 +665,6 @@ void TypeImpl<Config>::TypePrint(FILE* out, PrintDimension dim) {
     PrintF(out, ")");
   }
 }
-#endif


 template class TypeImpl<ZoneTypeConfig>;
Index: src/types.h
diff --git a/src/types.h b/src/types.h
index 42a50b922bd0f0f56441d853687b4fd390466041..0d172bdf0eb9bc64514bbb1fc7caf2f426dc0285 100644
--- a/src/types.h
+++ b/src/types.h
@@ -293,11 +293,9 @@ class TypeImpl : public Config::Base {
   static TypeHandle Convert(
       typename OtherTypeImpl::TypeHandle type, Region* region);

-#ifdef OBJECT_PRINT
   enum PrintDimension { BOTH_DIMS, SEMANTIC_DIM, REPRESENTATION_DIM };
   void TypePrint(PrintDimension = BOTH_DIMS);
   void TypePrint(FILE* out, PrintDimension = BOTH_DIMS);
-#endif

  private:
   template<class> friend class Iterator;
@@ -349,10 +347,8 @@ class TypeImpl : public Config::Base {
   static int ExtendIntersection(
UnionedHandle unioned, TypeHandle t, TypeHandle other, int current_size);

-#ifdef OBJECT_PRINT
   static const char* bitset_name(int bitset);
   static void BitsetTypePrint(FILE* out, int bitset);
-#endif
 };




--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to