From d8c9b5431b05550ee3a232e95af50a5e3d9ce4b5 Mon Sep 17 00:00:00 2001 From: Steve Reinhardt Date: Fri, 3 Jan 2014 17:08:43 -0800 Subject: [PATCH] python: provide better error message for wrapped C++ methods If you successfully export a C++ SimObject method, but try to invoke it from Python before the C++ object is created, you get a confusing error that says the attribute does not exist, making you question whether you successfully exported the method at all. In reality, your only problem is that you're calling the method too soon. This patch enhances the error message to give you a better clue. --- src/python/m5/SimObject.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/python/m5/SimObject.py b/src/python/m5/SimObject.py index 013f609d6..fc5416cbf 100644 --- a/src/python/m5/SimObject.py +++ b/src/python/m5/SimObject.py @@ -724,9 +724,15 @@ class SimObject(object): if self._ccObject and hasattr(self._ccObject, attr): return getattr(self._ccObject, attr) - raise AttributeError, "object '%s' has no attribute '%s'" \ + err_string = "object '%s' has no attribute '%s'" \ % (self.__class__.__name__, attr) + if not self._ccObject: + err_string += "\n (C++ object is not yet constructed," \ + " so wrapped C++ methods are unavailable.)" + + raise AttributeError, err_string + # Set attribute (called on foo.attr = value when foo is an # instance of class cls). def __setattr__(self, attr, value): -- 2.30.2