)]}'
{"functions":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"1cef25d0db7aaafefa9f83919339be3595deea8e","unresolved":false,"context_lines":[{"line_number":569,"context_line":"    shift 3"},{"line_number":570,"context_line":"    local values"},{"line_number":571,"context_line":"    for v in $@; do"},{"line_number":572,"context_line":"        # Reverse the values"},{"line_number":573,"context_line":"        values\u003d\"$v ${values}\""},{"line_number":574,"context_line":"    done"},{"line_number":575,"context_line":"    if ! grep -q \"^\\[$section\\]\" \"$file\"; then"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAN3%2F%2F%2FT8%3D","line":572,"updated":"2013-03-04 14:50:49.000000000","message":"It\u0027s clear what this is doing, but not why. Could you expand the comment to mention that the sed command below that adds the values to the file causes them to be added in the wrong order unless the argument list is reversed?","commit_id":"c2a968b112a7d3f1bea3d954dd7cc77cb6c63aa8"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"ecd01bdda27b43491a99b7b3b236f9bbb8203d33","unresolved":false,"context_lines":[{"line_number":569,"context_line":"    shift 3"},{"line_number":570,"context_line":"    local values"},{"line_number":571,"context_line":"    for v in $@; do"},{"line_number":572,"context_line":"        # Reverse the values"},{"line_number":573,"context_line":"        values\u003d\"$v ${values}\""},{"line_number":574,"context_line":"    done"},{"line_number":575,"context_line":"    if ! grep -q \"^\\[$section\\]\" \"$file\"; then"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAN3%2F%2F9oU%3D","line":572,"in_reply_to":"AAAAN3%2F%2F%2FT8%3D","updated":"2013-03-05 05:53:31.000000000","message":"Done","commit_id":"c2a968b112a7d3f1bea3d954dd7cc77cb6c63aa8"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"7f658f75cd64bd20fc0296b0fa2d2a8c54bd4dda","unresolved":false,"context_lines":[{"line_number":599,"context_line":"    local file\u003d$1"},{"line_number":600,"context_line":"    local section\u003d$2"},{"line_number":601,"context_line":"    local option\u003d$3"},{"line_number":602,"context_line":"    local values\u003d\"$(iniget_multiline $file $section $option) $4\""},{"line_number":603,"context_line":"    iniset_multiline $file $section $option $values"},{"line_number":604,"context_line":"}"},{"line_number":605,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAN3%2F%2F86o%3D","line":602,"updated":"2013-03-05 14:22:21.000000000","message":"Why not $@ instead of $4?","commit_id":"2d430864f2d82e3fcb52ae94e135289f61057a43"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"f131e0b08470e3cb971eac9f8d97a780c0c98999","unresolved":false,"context_lines":[{"line_number":599,"context_line":"    local file\u003d$1"},{"line_number":600,"context_line":"    local section\u003d$2"},{"line_number":601,"context_line":"    local option\u003d$3"},{"line_number":602,"context_line":"    local values\u003d\"$(iniget_multiline $file $section $option) $4\""},{"line_number":603,"context_line":"    iniset_multiline $file $section $option $values"},{"line_number":604,"context_line":"}"},{"line_number":605,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAN3%2F%2F7fI%3D","line":602,"in_reply_to":"AAAAN3%2F%2F86o%3D","updated":"2013-03-06 01:46:37.000000000","message":"Done","commit_id":"2d430864f2d82e3fcb52ae94e135289f61057a43"}]}