)]}'
{"swift/cli/recon.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"1118e12ca208f3dc02bd88528834c62efd9fb7f3","unresolved":false,"context_lines":[{"line_number":874,"context_line":"        args.add_option(\u0027--top\u0027, type\u003d\u0027int\u0027, metavar\u003d\u0027COUNT\u0027, default\u003d0,"},{"line_number":875,"context_line":"                        help\u003d\u0027Also show the top COUNT entries in rank order.\u0027)"},{"line_number":876,"context_line":"        args.add_option(\u0027--all\u0027, action\u003d\"store_true\","},{"line_number":877,"context_line":"                        help\u003d\"Perform all checks. Equal to \\t\\t\\t-arudlq \""},{"line_number":878,"context_line":"                        \"--md5 --sockstat --auditor --updater --expirer\")"},{"line_number":879,"context_line":"        args.add_option(\u0027--region\u0027, type\u003d\"int\","},{"line_number":880,"context_line":"                        help\u003d\"Only query servers in specified region\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_2896c34c","line":877,"updated":"2015-02-10 22:29:01.000000000","message":"I\u0027m thrilled that it works (and I tested it on various terminal sizes), but why not just do this:\n\n        args.add_option(\u0027--all\u0027, action\u003d\"store_true\",\n                        help\u003d\"Perform all checks. Equal to: swift-recon -arudlq \"\n                        \"--md5 --sockstat --auditor --updater --expirer\")","commit_id":"2e5eb7dbed6d35c12d2debeac02ec17f3f4c0783"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"6624f3440a96afab79046d71b760dbafe1ef6827","unresolved":false,"context_lines":[{"line_number":874,"context_line":"        args.add_option(\u0027--top\u0027, type\u003d\u0027int\u0027, metavar\u003d\u0027COUNT\u0027, default\u003d0,"},{"line_number":875,"context_line":"                        help\u003d\u0027Also show the top COUNT entries in rank order.\u0027)"},{"line_number":876,"context_line":"        args.add_option(\u0027--all\u0027, action\u003d\"store_true\","},{"line_number":877,"context_line":"                        help\u003d\"Perform all checks. Equal to \\t\\t\\t-arudlq \""},{"line_number":878,"context_line":"                        \"--md5 --sockstat --auditor --updater --expirer\")"},{"line_number":879,"context_line":"        args.add_option(\u0027--region\u0027, type\u003d\"int\","},{"line_number":880,"context_line":"                        help\u003d\"Only query servers in specified region\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_049f6e51","line":877,"in_reply_to":"da86d52c_184775a4","updated":"2015-02-23 00:46:39.000000000","message":"Looked at the code of textwrap and optparse, I think your approach is the best, only because it would involve a lot of hacking to handle the \u0027\\n\u0027 character which is being replaced with a \u0027 \u0027 in textwrap, unless we implement our own HelpFormatter and then recreate a version of the wrap method in textwrap.\n\nYour approach works on every console as textwrap uses an text_width so \"should\" always wrap at the same point.","commit_id":"2e5eb7dbed6d35c12d2debeac02ec17f3f4c0783"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"7e21601f731e8aa94ff6df7183c1db12734aead9","unresolved":false,"context_lines":[{"line_number":874,"context_line":"        args.add_option(\u0027--top\u0027, type\u003d\u0027int\u0027, metavar\u003d\u0027COUNT\u0027, default\u003d0,"},{"line_number":875,"context_line":"                        help\u003d\u0027Also show the top COUNT entries in rank order.\u0027)"},{"line_number":876,"context_line":"        args.add_option(\u0027--all\u0027, action\u003d\"store_true\","},{"line_number":877,"context_line":"                        help\u003d\"Perform all checks. Equal to \\t\\t\\t-arudlq \""},{"line_number":878,"context_line":"                        \"--md5 --sockstat --auditor --updater --expirer\")"},{"line_number":879,"context_line":"        args.add_option(\u0027--region\u0027, type\u003d\"int\","},{"line_number":880,"context_line":"                        help\u003d\"Only query servers in specified region\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_184775a4","line":877,"in_reply_to":"da86d52c_2896c34c","updated":"2015-02-18 12:52:16.000000000","message":"@Pete: That was my first try, but now you have two lines, and I wanted to have a single line for copy-pasting.\nInterestingly adding a \u0027\\n\u0027 after \"Equal to:\" doesn\u0027t work.","commit_id":"2e5eb7dbed6d35c12d2debeac02ec17f3f4c0783"}]}
