)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"782146f470a90911c2bdb77dc90ad708ef2a4914","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     sue \u003csugar-2008@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-08-12 18:18:23 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ValueError shen passing a string representation of a float into int"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the QemuImgInfo class, the magnitude could be a string representation"},{"line_number":10,"context_line":"of a float, which would leads to \"ValueError: invalid literal for int()...\""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a926be0c_26f332ba","line":7,"range":{"start_line":7,"start_character":11,"end_line":7,"end_character":15},"updated":"2021-08-12 12:06:44.000000000","message":"when","commit_id":"f37f63073dd33bde6c81899ae00992e73936c086"}],"oslo_utils/imageutils.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"782146f470a90911c2bdb77dc90ad708ef2a4914","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    BACKING_FILE_RE \u003d re.compile((r\"^(.*?)\\s*\\(actual\\s+path\\s*:\""},{"line_number":50,"context_line":"                                  r\"\\s+(.*?)\\)\\s*$\"), re.I)"},{"line_number":51,"context_line":"    TOP_LEVEL_RE \u003d re.compile(r\"^([\\w\\d\\s\\_\\-]+):(.*)$\")"},{"line_number":52,"context_line":"    SIZE_RE \u003d re.compile(r\"([0-9]+[eE][-+][0-9]+|\\d*\\.?\\d+)\""},{"line_number":53,"context_line":"                         r\"\\s*(\\w+)?(\\s*\\(\\s*(\\d+)\\s+bytes\\s*\\))?\","},{"line_number":54,"context_line":"                         re.I)"},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"05411bd1_b63053f0","line":52,"range":{"start_line":52,"start_character":27,"end_line":52,"end_character":59},"updated":"2021-08-12 12:06:44.000000000","message":"this is group 1...","commit_id":"f37f63073dd33bde6c81899ae00992e73936c086"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"782146f470a90911c2bdb77dc90ad708ef2a4914","unresolved":false,"context_lines":[{"line_number":50,"context_line":"                                  r\"\\s+(.*?)\\)\\s*$\"), re.I)"},{"line_number":51,"context_line":"    TOP_LEVEL_RE \u003d re.compile(r\"^([\\w\\d\\s\\_\\-]+):(.*)$\")"},{"line_number":52,"context_line":"    SIZE_RE \u003d re.compile(r\"([0-9]+[eE][-+][0-9]+|\\d*\\.?\\d+)\""},{"line_number":53,"context_line":"                         r\"\\s*(\\w+)?(\\s*\\(\\s*(\\d+)\\s+bytes\\s*\\))?\","},{"line_number":54,"context_line":"                         re.I)"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    def __init__(self, cmd_output\u003dNone, format\u003d\u0027human\u0027):"}],"source_content_type":"text/x-python","patch_set":2,"id":"33402239_1a394776","line":53,"range":{"start_line":53,"start_character":30,"end_line":53,"end_character":36},"updated":"2021-08-12 12:06:44.000000000","message":"...and this is group 2","commit_id":"f37f63073dd33bde6c81899ae00992e73936c086"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"782146f470a90911c2bdb77dc90ad708ef2a4914","unresolved":true,"context_lines":[{"line_number":123,"context_line":"        if bytes_info:"},{"line_number":124,"context_line":"            return int(real_size.group(4))"},{"line_number":125,"context_line":"        elif not unit_of_measure:"},{"line_number":126,"context_line":"            return int(float(magnitude))"},{"line_number":127,"context_line":"        # Allow abbreviated unit such as K to mean KB for compatibility."},{"line_number":128,"context_line":"        if len(unit_of_measure) \u003d\u003d 1 and unit_of_measure !\u003d \u0027B\u0027:"},{"line_number":129,"context_line":"            unit_of_measure +\u003d \u0027B\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"a6950509_fb556a27","line":126,"updated":"2021-08-12 12:06:44.000000000","message":"Based on this above, this code path should not execute:\n\n  \u003e\u003e\u003e x \u003d re.compile(\u0027([0-9]+[eE][-+][0-9]+|\\\\d*\\\\.?\\\\d+)\\\\s*(\\\\w+)?(\\\\s*\\\\(\\\\s*(\\\\d+)\\\\s+bytes\\\\s*\\\\))?\u0027, re.IGNORECASE)\n  \u003e\u003e\u003e y \u003d x.search(\u002739.2 MiB (41126400 bytes)\u0027)\n  \u003e\u003e\u003e y.group(1)  # magnitude\n  39.2\n  \u003e\u003e\u003e y.group(2)  # unit_of_measure\n  \u0027MiB\u0027\n\nSo \u0027not unit_of_measure\u0027 should be False.\n\nSafe to say, I\u0027m not entirely sure what\u0027s going on here but it seems obvious that we need a unit test that proves that this fix does what it says it does. This test should fail without this change and pass with it. Can you add it?","commit_id":"f37f63073dd33bde6c81899ae00992e73936c086"}]}