recipe engine: remove Path.default_tostring_fn

recipe_engine/config_types.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import abc
 import re
 
 from collections import namedtuple
 
 from . import types
@@ skipping 41 matching lines @@
     cls = self.__class__
     return self._TOSTRING_MAP.get(cls.__name__, cls.default_tostring_fn)
 
   @classmethod
   def set_tostring_fn(cls, new_tostring_fn):
     assert cls.__name__ not in cls._TOSTRING_MAP, (
         'tostring_fn already installed for %s' % cls)
     cls._TOSTRING_MAP[cls.__name__] = new_tostring_fn
 
   def default_tostring_fn(self):
-    raise NotImplementedError
+    raise NotImplementedError()
 
   def __str__(self):
     return self.tostring_fn(self) # pylint: disable=not-callable
 
 
 class BasePath(object):
   __metaclass__ = abc.ABCMeta
 
 
 class NamedBasePath(BasePath, namedtuple('NamedBasePath', 'name')):
@@ skipping 80 matching lines @@
     """True if |child| is in a subdirectory of this path."""
     # Assumes base paths are not nested.
     # TODO(vadimsh): We should not rely on this assumption.
     if self.base != child.base:
       return False
     # A path is not a parent to itself.
     if len(self.pieces) >= len(child.pieces):
       return False
     return child.pieces[:len(self.pieces)] == self.pieces
 
-  def default_tostring_fn(self):
-    suffix = ''
-    if self.platform_ext:
-      suffix = ', platform_ext=%r' % (self.platform_ext,)
-    pieces = ''
-    if self.pieces:
-      pieces = ', ' + (', '.join(map(repr, self.pieces)))
-    return 'Path(\'%s\'%s%s)' % (self.base, pieces, suffix)
-
   def __repr__(self):
     s = "Path(%r" % self.base
     if self.pieces:
       s += ", %s" % ",".join(repr(x) for x in self.pieces)
 
     return s + ")"