)]}'
{"ceilometer/api/controllers/v2.py":[{"author":{"_account_id":8005,"name":"Ryan Petrello","email":"ryan@ryanpetrello.com","username":"ryanpetrello"},"change_message_id":"9dbcb1ba32f19784ab841a38f80f4f01132fd12c","unresolved":false,"context_lines":[{"line_number":705,"context_line":"                   )"},{"line_number":706,"context_line":""},{"line_number":707,"context_line":""},{"line_number":708,"context_line":"class JSonRenderer(object):"},{"line_number":709,"context_line":"    @staticmethod"},{"line_number":710,"context_line":"    def __init__(path, extra_vars):"},{"line_number":711,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2Fd8E%3D","line":708,"updated":"2014-01-13 15:48:47.000000000","message":"Why is this necessary vs the vanilla `json` renderer provided by pecan?","commit_id":"2aaf4e4f781efd85b5aa418a5c8d2fa539e62014"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"190506a4ea1939c35bedd305f8a6bf7e394ad815","unresolved":false,"context_lines":[{"line_number":705,"context_line":"                   )"},{"line_number":706,"context_line":""},{"line_number":707,"context_line":""},{"line_number":708,"context_line":"class JSonRenderer(object):"},{"line_number":709,"context_line":"    @staticmethod"},{"line_number":710,"context_line":"    def __init__(path, extra_vars):"},{"line_number":711,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2FYLw%3D","line":708,"in_reply_to":"AAAAUH%2F%2Fd8E%3D","updated":"2014-01-14 13:06:23.000000000","message":"I don\u0027t know. I probably didn\u0027t read the doc enough, I\u0027ve just stole that code from WSME. Pardon me. :)","commit_id":"2aaf4e4f781efd85b5aa418a5c8d2fa539e62014"},{"author":{"_account_id":8005,"name":"Ryan Petrello","email":"ryan@ryanpetrello.com","username":"ryanpetrello"},"change_message_id":"9dbcb1ba32f19784ab841a38f80f4f01132fd12c","unresolved":false,"context_lines":[{"line_number":723,"context_line":"    generic\u003dFalse)"},{"line_number":724,"context_line":""},{"line_number":725,"context_line":""},{"line_number":726,"context_line":"def colander_expose(f):"},{"line_number":727,"context_line":"    @functools.wraps(f)"},{"line_number":728,"context_line":"    def callfunction(*args, **kwargs):"},{"line_number":729,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2Fd54%3D","line":726,"updated":"2014-01-13 15:48:47.000000000","message":"The way that WSME accomplished this in pecan by wrapping pecan\u0027s @expose (especially multiple times) always sort of grossed me out.  Is there a reason for doing it?  Why not just have two decorators?  Seems easier to read/grok to me:\n\n@colander_exception_handling_bits\n@pecan.expose(\u0027...\u0027)\ndef post(self, ...):\n    pass","commit_id":"2aaf4e4f781efd85b5aa418a5c8d2fa539e62014"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"e80c9df53541987201014117df1ecf1e452f5b51","unresolved":false,"context_lines":[{"line_number":723,"context_line":"    generic\u003dFalse)"},{"line_number":724,"context_line":""},{"line_number":725,"context_line":""},{"line_number":726,"context_line":"def colander_expose(f):"},{"line_number":727,"context_line":"    @functools.wraps(f)"},{"line_number":728,"context_line":"    def callfunction(*args, **kwargs):"},{"line_number":729,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2Fd2M%3D","line":726,"in_reply_to":"AAAAUH%2F%2Fd54%3D","updated":"2014-01-13 15:53:52.000000000","message":"Having to remember to add the exception handling, and get the decorators in the right order, seems like an opportunity for devs less familiar with the internals of pecan and colander to get it wrong, so I like the idea of providing a special decorator to handle the integration.","commit_id":"2aaf4e4f781efd85b5aa418a5c8d2fa539e62014"}]}