)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"7302e3c0ebfe9b7d94d6b34af46966d2542e5047","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Lucian Petrut \u003clpetrut@cloudbasesolutions.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-06-23 18:49:15 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Perform case insensitive comparisons when dealing with Windows paths"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"While Windows paths are case insensitive, some checks performed in the"},{"line_number":10,"context_line":"SMBFS driver as well as its parent class are not, breaking the driver\u0027s"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f201791_381bd381","line":7,"updated":"2017-06-23 15:53:55.000000000","message":"Nit: If you do another update, please try to keep this in the 50 character limit.","commit_id":"210262d2b6f930ac276ba4ff5f633663936e24d5"}],"cinder/utils.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"a792e26d83935657ed4c4eda3661f14410c22f3a","unresolved":false,"context_lines":[{"line_number":1132,"context_line":"            if k and k.startswith(prefix)}"},{"line_number":1133,"context_line":""},{"line_number":1134,"context_line":""},{"line_number":1135,"context_line":"def paths_equal(path_a, path_b):"},{"line_number":1136,"context_line":"    if sys.platform \u003d\u003d \u0027win32\u0027:"},{"line_number":1137,"context_line":"        # Windows paths are case insensitive."},{"line_number":1138,"context_line":"        return path_a.lower() \u003d\u003d path_b.lower()"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_157d82e6","line":1135,"updated":"2017-06-23 15:09:10.000000000","message":"This can use os.path.normcase() and skip the platform check.\n\nIt also should probably have a more specific name -- it sounds like a method that would compare using something like normpath(), but it doesn\u0027t go that far.","commit_id":"13a024fc01beeebbbe3c665a954f4b7bac48b2bc"},{"author":{"_account_id":8543,"name":"Lucian Petrut","email":"lpetrut@cloudbasesolutions.com","username":"plucian"},"change_message_id":"fdd6ef89395fd794f4713f5a6fee6532eaa97bc5","unresolved":false,"context_lines":[{"line_number":1132,"context_line":"            if k and k.startswith(prefix)}"},{"line_number":1133,"context_line":""},{"line_number":1134,"context_line":""},{"line_number":1135,"context_line":"def paths_equal(path_a, path_b):"},{"line_number":1136,"context_line":"    if sys.platform \u003d\u003d \u0027win32\u0027:"},{"line_number":1137,"context_line":"        # Windows paths are case insensitive."},{"line_number":1138,"context_line":"        return path_a.lower() \u003d\u003d path_b.lower()"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_b86da333","line":1135,"in_reply_to":"5f201791_157d82e6","updated":"2017-06-23 15:44:55.000000000","message":"Nice, thanks for mentioning os.path.normcase.\n\nAbout the naming, indeed, I was a bit afraid that the name would imply that. I guess I could just name it \u0027path_normcase_equal\u0027, suggesting that we only care about the case.","commit_id":"13a024fc01beeebbbe3c665a954f4b7bac48b2bc"}]}