)]}'
{"ceilometer/compute/manager.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"2522e1a057c65d5ba6d80eba6e772766d74c4825","unresolved":false,"context_lines":[{"line_number":101,"context_line":"                self.poll_instance(context, instance)"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    @property"},{"line_number":104,"context_line":"    def inspector(self):"},{"line_number":105,"context_line":"        return self._inspector"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F25I%3D","line":104,"updated":"2012-12-12 10:21:21.000000000","message":"What\u0027s the point of using a property here? Being read-only?","commit_id":"9c8a8e3caafb51efbfeee2f592e00f681c4cd171"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"825c49027d8f6df8918c1c238301e177a1829433","unresolved":false,"context_lines":[{"line_number":101,"context_line":"                self.poll_instance(context, instance)"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    @property"},{"line_number":104,"context_line":"    def inspector(self):"},{"line_number":105,"context_line":"        return self._inspector"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F%2FoA%3D","line":104,"in_reply_to":"AAAAMX%2F%2F0Co%3D","updated":"2012-12-17 10:53:32.000000000","message":"Yeah, my point was why not store things in self.inspector directly actually, that\u0027s why I asked if it was to be read-only. So your answer seems to be yes, that\u0027s fine with me, just wanted to be sure.","commit_id":"9c8a8e3caafb51efbfeee2f592e00f681c4cd171"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"431ce2d0763bd7269a9ee7079f9fc6a5a5f075b5","unresolved":false,"context_lines":[{"line_number":101,"context_line":"                self.poll_instance(context, instance)"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    @property"},{"line_number":104,"context_line":"    def inspector(self):"},{"line_number":105,"context_line":"        return self._inspector"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F0Co%3D","line":104,"in_reply_to":"AAAAMX%2F%2F25I%3D","updated":"2012-12-14 14:37:27.000000000","message":"To allows the attribute-style usage, i.e. manager.inspector as opposed to manager.inspector(), while disallowing an external assignment.\n\nSure the _inspector attribute can be assigned externally, but if the calling code starts messing around with attributes named with a leading underscore, then all bets are off.","commit_id":"9c8a8e3caafb51efbfeee2f592e00f681c4cd171"}]}