)]}'
{"tvix/cli/src/known_paths.rs":[{"author":{"_account_id":1000036,"name":"flokli","email":"flokli@flokli.de","username":"flokli"},"change_message_id":"a286e0e865ab0ca32c78c63e23c362992d762cd6","unresolved":true,"context_lines":[{"line_number":164,"context_line":"    }"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    pub fn add_replacement_string\u003cD: ToString\u003e(\u0026mut self, drv: D, replacement_str: \u0026str) {"},{"line_number":167,"context_line":"        #[allow(unused_variables)] // assertions on this only compiled in debug builds"},{"line_number":168,"context_line":"        let old \u003d self"},{"line_number":169,"context_line":"            .replacements"},{"line_number":170,"context_line":"            .insert(drv.to_string(), replacement_str.to_owned());"}],"source_content_type":"text/x-rustsrc","patch_set":1,"id":"31e0bb36_d51a9763","line":167,"updated":"2023-03-11 22:33:15.000000000","message":"Can\u0027t you just move the `let old \u003d` inside the `#[cfg(debug_assertions)]` scope?","commit_id":"4acdc44685dd091e9e6dac9699006d4ae9c0d518"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"74cf3b6b0c9de8f573b0366567039eb11d264408","unresolved":true,"context_lines":[{"line_number":164,"context_line":"    }"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    pub fn add_replacement_string\u003cD: ToString\u003e(\u0026mut self, drv: D, replacement_str: \u0026str) {"},{"line_number":167,"context_line":"        #[allow(unused_variables)] // assertions on this only compiled in debug builds"},{"line_number":168,"context_line":"        let old \u003d self"},{"line_number":169,"context_line":"            .replacements"},{"line_number":170,"context_line":"            .insert(drv.to_string(), replacement_str.to_owned());"}],"source_content_type":"text/x-rustsrc","patch_set":1,"id":"aa10e0f9_20e86dfd","line":167,"in_reply_to":"31e0bb36_d51a9763","updated":"2023-03-11 22:43:29.000000000","message":"You mean something like duplicate the actual code into two blocks, and fence once against `cfg(not(debug_assertions))`, and the other one as is?\n\nPersonally I find this more readable, the compiler will optimise the unused binding away anyways.","commit_id":"4acdc44685dd091e9e6dac9699006d4ae9c0d518"},{"author":{"_account_id":1000036,"name":"flokli","email":"flokli@flokli.de","username":"flokli"},"change_message_id":"d4bcc9cabb8009cce8571202ea405e00c347e27c","unresolved":false,"context_lines":[{"line_number":164,"context_line":"    }"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    pub fn add_replacement_string\u003cD: ToString\u003e(\u0026mut self, drv: D, replacement_str: \u0026str) {"},{"line_number":167,"context_line":"        #[allow(unused_variables)] // assertions on this only compiled in debug builds"},{"line_number":168,"context_line":"        let old \u003d self"},{"line_number":169,"context_line":"            .replacements"},{"line_number":170,"context_line":"            .insert(drv.to_string(), replacement_str.to_owned());"}],"source_content_type":"text/x-rustsrc","patch_set":1,"id":"cd0c1a46_315b9a2e","line":167,"in_reply_to":"aa10e0f9_20e86dfd","updated":"2023-03-11 22:53:48.000000000","message":"Ok","commit_id":"4acdc44685dd091e9e6dac9699006d4ae9c0d518"}]}
