)]}'
{"cinder/utils.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"208addfe0ad27e92c543a4db700c1bc2b5419627","unresolved":false,"context_lines":[{"line_number":164,"context_line":"    if not metadata:"},{"line_number":165,"context_line":"        metadata \u003d {}"},{"line_number":166,"context_line":"    if not isinstance(metadata, dict):"},{"line_number":167,"context_line":"        msg \u003d _(\"Metadata should be a dict.\")"},{"line_number":168,"context_line":"        raise exception.InvalidInput(msg)"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    for k, v in metadata.items():"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_533096e8","line":167,"range":{"start_line":167,"start_character":8,"end_line":167,"end_character":11},"updated":"2017-06-26 06:07:05.000000000","message":"This is fine, but for future reference it\u0027s better not to assign to a temporary variable if it\u0027s only used for one log call or one exception constructor. It would be better to put the string right in the InvalidInput() creation.","commit_id":"bf775499f1bded8c487d39888f61e85aca125fd8"}]}