)]}'
{"quantum/extensions/securitygroup.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"c7e3c66d69e52b40f804099f34cce07f5f40c363","unresolved":false,"context_lines":[{"line_number":101,"context_line":"        return"},{"line_number":102,"context_line":"    try:"},{"line_number":103,"context_line":"        val \u003d int(value)"},{"line_number":104,"context_line":"        if val \u003e\u003d 0 and val \u003c\u003d 255:"},{"line_number":105,"context_line":"            return val"},{"line_number":106,"context_line":"        raise SecurityGroupRuleInvalidProtocol("},{"line_number":107,"context_line":"            protocol\u003dvalue, values\u003dsg_supported_protocols)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2F4A0%3D","line":104,"updated":"2013-06-06 22:40:29.000000000","message":"if 0 \u003c\u003d val \u003c\u003d 255:","commit_id":"8708bc538e2515899cf874e1e5e2275360fb5ef6"},{"author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"unresolved":false,"context_lines":[{"line_number":101,"context_line":"        return"},{"line_number":102,"context_line":"    try:"},{"line_number":103,"context_line":"        val \u003d int(value)"},{"line_number":104,"context_line":"        if val \u003e\u003d 0 and val \u003c\u003d 255:"},{"line_number":105,"context_line":"            return val"},{"line_number":106,"context_line":"        raise SecurityGroupRuleInvalidProtocol("},{"line_number":107,"context_line":"            protocol\u003dvalue, values\u003dsg_supported_protocols)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2F38s%3D","line":104,"in_reply_to":"AAAAPX%2F%2F3%2BY%3D","updated":"2013-06-06 23:14:19.000000000","message":"Probably be better the other way, but I don\u0027t see any reason to update the patch just for this. If someone else finds something they don\u0027t like, I guess change it then?","commit_id":"8708bc538e2515899cf874e1e5e2275360fb5ef6"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"unresolved":false,"context_lines":[{"line_number":101,"context_line":"        return"},{"line_number":102,"context_line":"    try:"},{"line_number":103,"context_line":"        val \u003d int(value)"},{"line_number":104,"context_line":"        if val \u003e\u003d 0 and val \u003c\u003d 255:"},{"line_number":105,"context_line":"            return val"},{"line_number":106,"context_line":"        raise SecurityGroupRuleInvalidProtocol("},{"line_number":107,"context_line":"            protocol\u003dvalue, values\u003dsg_supported_protocols)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2F1kc%3D","line":104,"in_reply_to":"AAAAPX%2F%2F38s%3D","updated":"2013-06-07 21:38:10.000000000","message":"No need to revise the patch for a minor stylistic equivalent.","commit_id":"8708bc538e2515899cf874e1e5e2275360fb5ef6"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"c0145a7c9da7518ee09c7d8924cec7772abe5b85","unresolved":false,"context_lines":[{"line_number":101,"context_line":"        return"},{"line_number":102,"context_line":"    try:"},{"line_number":103,"context_line":"        val \u003d int(value)"},{"line_number":104,"context_line":"        if val \u003e\u003d 0 and val \u003c\u003d 255:"},{"line_number":105,"context_line":"            return val"},{"line_number":106,"context_line":"        raise SecurityGroupRuleInvalidProtocol("},{"line_number":107,"context_line":"            protocol\u003dvalue, values\u003dsg_supported_protocols)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2F3%2BY%3D","line":104,"in_reply_to":"AAAAPX%2F%2F4A0%3D","updated":"2013-06-06 22:52:49.000000000","message":"I\u0027d prefer to leave it like this. Line 133 already does it this way. I\u0027ll change it though if you prefer :)","commit_id":"8708bc538e2515899cf874e1e5e2275360fb5ef6"}]}