)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"7cb5d452b73934aa69c6342055b8bb9117b2773f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"To modify a format and logic errors in codes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"First,there have a format question: python is strictly comply"},{"line_number":10,"context_line":"with indentation format,obviously ,in this code,the sentence else"},{"line_number":11,"context_line":"and if are not in the same columns;"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Secondly,there have a logical problem in the code:"},{"line_number":14,"context_line":"if the \"if\" statement:"},{"line_number":15,"context_line":"\"if (image_meta [\u0027 owner \u0027]! \u003d volume [\u0027 project_id] and"},{"line_number":16,"context_line":"image_meta [\u0027 owner \u0027]! \u003d image_owner)\""},{"line_number":17,"context_line":"has been failed until it has been completed,then sentence \"continue\""},{"line_number":18,"context_line":"will be implement uninterrupted until run the sentence"},{"line_number":19,"context_line":"\" (_LI (\" Will clone a volume from the image (id) volume % s. \"),"},{"line_number":20,"context_line":"  {\" id \": image_volume. Id})\""},{"line_number":21,"context_line":"and then it Will be break , at this time,he has mistakened believe that"},{"line_number":22,"context_line":"he has found the legal image and ready to perform the copy sentence,however,"},{"line_number":23,"context_line":"the reality is he has \u0027continue\u0027 all the judgment statement,and have not"},{"line_number":24,"context_line":"been found the legal image."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"This patch have improved this logical probelem."},{"line_number":27,"context_line":"logic errors."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Change-Id: I4573c144b5cad1db45aa5ac608787d6b00a694a2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5a8bc5a2_5c10144f","line":27,"range":{"start_line":9,"start_character":0,"end_line":27,"end_character":13},"updated":"2016-09-22 09:07:11.000000000","message":"please reorganise and shorten your commit messages,code is enough","commit_id":"2e6fa4cc73fceafe93ca10ab322a947371f4fd28"}],"cinder/volume/flows/manager/create_volume.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"7cb5d452b73934aa69c6342055b8bb9117b2773f","unresolved":false,"context_lines":[{"line_number":621,"context_line":"                             \"%(id)s.\"), {\u0027id\u0027: image_volume.id})"},{"line_number":622,"context_line":"                accessible_image_volume \u003d True"},{"line_number":623,"context_line":"                break"},{"line_number":624,"context_line":"            else:"},{"line_number":625,"context_line":"                LOG.info(_LI(\"Skipping image volume %(id)s because \""},{"line_number":626,"context_line":"                             \"it is not accessible by current Tenant.\"),"},{"line_number":627,"context_line":"                         {\u0027id\u0027: image_volume.id})"},{"line_number":628,"context_line":"                continue"},{"line_number":629,"context_line":""},{"line_number":630,"context_line":"        if not accessible_image_volume:"},{"line_number":631,"context_line":"            LOG.debug(\"No accessible image volume for image %(id)s found.\","}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_dc0984df","line":628,"range":{"start_line":624,"start_character":11,"end_line":628,"end_character":24},"updated":"2016-09-22 09:07:11.000000000","message":"continue\u0026else are redundant?","commit_id":"2e6fa4cc73fceafe93ca10ab322a947371f4fd28"}]}