)]}'
{"nova/console/api.py":[{"author":{"_account_id":357,"name":"Devin Carlen","email":"devin@openstack.org","username":"devcamcar"},"change_message_id":"0de4a66f9cc5f429f58be22ccb32fe4d4c407cfd","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        return rpc.call(context, topic, {\u0027method\u0027: \u0027get_console_topic\u0027,"},{"line_number":74,"context_line":"                                         \u0027args\u0027: {\u0027fake\u0027: 1}})"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def _translate_uuid_if_necessary(self, context, instance_id):"},{"line_number":77,"context_line":"        if utils.is_uuid_like(instance_id):"},{"line_number":78,"context_line":"            instance \u003d self.db.instance_get_by_uuid(context, instance_id)"},{"line_number":79,"context_line":"            instance_id \u003d instance[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAACX%2F%2F6Cw%3D","line":76,"updated":"2011-10-27 23:58:18.000000000","message":"Just a nit, but can we call this _maybe_translate_uuid instead?  Would match existing patterns.","commit_id":"fa74f5c59da8e1641d97333d0b8ee6147fb17947"},{"author":{"_account_id":357,"name":"Devin Carlen","email":"devin@openstack.org","username":"devcamcar"},"change_message_id":"1c8e3b86145af365d5c295557f28befa01af511e","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        return rpc.call(context, topic, {\u0027method\u0027: \u0027get_console_topic\u0027,"},{"line_number":74,"context_line":"                                         \u0027args\u0027: {\u0027fake\u0027: 1}})"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def _translate_uuid_if_necessary(self, context, instance_id):"},{"line_number":77,"context_line":"        if utils.is_uuid_like(instance_id):"},{"line_number":78,"context_line":"            instance \u003d self.db.instance_get_by_uuid(context, instance_id)"},{"line_number":79,"context_line":"            instance_id \u003d instance[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAACX%2F%2F5yA%3D","line":76,"in_reply_to":"AAAACX%2F%2F5%2BU%3D","updated":"2011-10-28 22:57:05.000000000","message":"Touche!  I\u0027ve seen it in other OpenStack projects, like in the dashboard and a few other places.  I think it\u0027s more succinct.","commit_id":"fa74f5c59da8e1641d97333d0b8ee6147fb17947"},{"author":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"change_message_id":"8987497d742e638cd915a81bdf842c075d02d44d","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        return rpc.call(context, topic, {\u0027method\u0027: \u0027get_console_topic\u0027,"},{"line_number":74,"context_line":"                                         \u0027args\u0027: {\u0027fake\u0027: 1}})"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def _translate_uuid_if_necessary(self, context, instance_id):"},{"line_number":77,"context_line":"        if utils.is_uuid_like(instance_id):"},{"line_number":78,"context_line":"            instance \u003d self.db.instance_get_by_uuid(context, instance_id)"},{"line_number":79,"context_line":"            instance_id \u003d instance[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAACX%2F%2F5%2BU%3D","line":76,"in_reply_to":"AAAACX%2F%2F6Cw%3D","updated":"2011-10-28 13:41:18.000000000","message":"Thanks for the feedback!\n\nI would love to use a standard naming scheme here, as I wasn\u0027t thrilled by this function name. However--I must be missing something--in my checkout I don\u0027t see any functions with the word \"maybe\" in them:\n\n$ grep -rn \"def.*maybe\" nova/ \n\n$\n\nCan you point me in the direction of where this or other similar patterns exist?\n\nThanks again.","commit_id":"fa74f5c59da8e1641d97333d0b8ee6147fb17947"}]}