)]}'
{"lib/neutron":[{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"d10f3135bf70cf3f1e7bc11b4785547aa1bfa205","unresolved":false,"context_lines":[{"line_number":590,"context_line":"        local f"},{"line_number":591,"context_line":"        for (( f\u003d0; $f \u003c ${#Q_PLUGIN_EXTRA_CONF_FILES[@]}; f+\u003d1 )); do"},{"line_number":592,"context_line":"            Q_PLUGIN_EXTRA_CONF_FILES[$f]\u003d$Q_PLUGIN_EXTRA_CONF_PATH/${Q_PLUGIN_EXTRA_CONF_FILES[$f]}"},{"line_number":593,"context_line":"            cp $NEUTRON_DIR/${Q_PLUGIN_EXTRA_CONF_FILES[$f]} /${Q_PLUGIN_EXTRA_CONF_FILES[$f]}"},{"line_number":594,"context_line":"        done"},{"line_number":595,"context_line":"    fi"},{"line_number":596,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAVX%2F%2FpWo%3D","side":"PARENT","line":593,"updated":"2014-03-02 02:35:42.000000000","message":"Why only remove the mkdir and the cp? The rest of this loop and the entire if block is pointless without these.","commit_id":"0ed4af02da0bd4a0f757dd8c2156913e6c7a724c"},{"author":{"_account_id":9596,"name":"Shashank Hegde","email":"shashank@aristanetworks.com","username":"shashank"},"change_message_id":"5b6a7f4f1001caecc6b7ade5d9befa423ebc625b","unresolved":false,"context_lines":[{"line_number":590,"context_line":"        local f"},{"line_number":591,"context_line":"        for (( f\u003d0; $f \u003c ${#Q_PLUGIN_EXTRA_CONF_FILES[@]}; f+\u003d1 )); do"},{"line_number":592,"context_line":"            Q_PLUGIN_EXTRA_CONF_FILES[$f]\u003d$Q_PLUGIN_EXTRA_CONF_PATH/${Q_PLUGIN_EXTRA_CONF_FILES[$f]}"},{"line_number":593,"context_line":"            cp $NEUTRON_DIR/${Q_PLUGIN_EXTRA_CONF_FILES[$f]} /${Q_PLUGIN_EXTRA_CONF_FILES[$f]}"},{"line_number":594,"context_line":"        done"},{"line_number":595,"context_line":"    fi"},{"line_number":596,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAVn%2F%2F2d4%3D","side":"PARENT","line":593,"in_reply_to":"AAAAVX%2F%2FpWo%3D","updated":"2014-03-03 19:57:18.000000000","message":"The loop is used to construct the absolute path to the conf file.\n\nIn the localrc, the variables are defined as:\nQ_PLUGIN_EXTRA_CONF_PATH\u003d/path/to/conf/files\nQ_PLUGIN_EXTRA_CONF_FILES\u003d(file1.conf, file2.conf)\n\nThe loop iterates through the Q_PLUGIN_EXTRA_CONF_FILES and constructs absolute path to the files.","commit_id":"0ed4af02da0bd4a0f757dd8c2156913e6c7a724c"}]}