)]}'
{"examples/common.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"1d4ded92054331310b30c8c8ef62f80eaa5e80cf","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"def get_data_option(opts):"},{"line_number":70,"context_line":"    if not opts.data:"},{"line_number":71,"context_line":"        return opts.data"},{"line_number":72,"context_line":"    try:"},{"line_number":73,"context_line":"        iddy \u003d uuid.UUID(opts.data)"},{"line_number":74,"context_line":"        return {\u0027id\u0027: iddy}"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_b6b0c3de","line":71,"updated":"2015-02-01 14:30:09.000000000","message":"I don\u0027t completely understand the purpose here, but why doesn\u0027t this return \u0027\u0027 empty string?","commit_id":"0a235978f8c1f3b3a057e38bffd4c09c267c1994"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"23ca2d34d35b457387ef6b57ac94fffbe8252095","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"def get_data_option(opts):"},{"line_number":70,"context_line":"    if not opts.data:"},{"line_number":71,"context_line":"        return opts.data"},{"line_number":72,"context_line":"    try:"},{"line_number":73,"context_line":"        iddy \u003d uuid.UUID(opts.data)"},{"line_number":74,"context_line":"        return {\u0027id\u0027: iddy}"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_f63ebb28","line":71,"in_reply_to":"da86d52c_76048b7e","updated":"2015-02-01 14:47:11.000000000","message":"I should be more explicit: opts.data is {} by default, so when you don\u0027t set --data you get that {} and this now works whether you set the right data or no data.\n\nI could make it return {} instead of opts.data if you\u0027d prefer.","commit_id":"0a235978f8c1f3b3a057e38bffd4c09c267c1994"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"5e71e33eb561313317db0be575746894d2cc97c5","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"def get_data_option(opts):"},{"line_number":70,"context_line":"    if not opts.data:"},{"line_number":71,"context_line":"        return opts.data"},{"line_number":72,"context_line":"    try:"},{"line_number":73,"context_line":"        iddy \u003d uuid.UUID(opts.data)"},{"line_number":74,"context_line":"        return {\u0027id\u0027: iddy}"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_76048b7e","line":71,"in_reply_to":"da86d52c_b6b0c3de","updated":"2015-02-01 14:40:55.000000000","message":"I guess it could do that, but this was the smaller change. If it returns an empty string we\u0027d have to make another change in the example scripts because we assume this function will always return them a dictionary, which it does in the other cases, because we then put it through something like \"cls.existing(**this_return_value)\"\n\nI noticed it a while ago but thought I was doing something wrong, and then when I was testing out that LimitsAbsolute change I noticed what it was doing.","commit_id":"0a235978f8c1f3b3a057e38bffd4c09c267c1994"}]}