3 ms·
What a weird statement. It's beautiful! You don't need comments, just read the code. No need to litter code with ambiguous language. The only place a comment fi
by porsager 3y ago
What a weird statement. It's beautiful! You don't need comments, just read the code. No need to litter code with ambiguous language. The only place a comment fits is if the purpose is not clear from the code.
- Thaxll 3y agoThis file does not need comments? https://github.com/zed-industries/zed/blob/main/crates/multi_buffer/src/multi_buffer.rs https://github.com/zed-industries/zed/blob/main/crates/multi...
- porsager 3y agoHonestly no. Not more at least. There are some that does exactly as I mention by mentioning things not in the code like: ``` // Non-generic part of edit, hoisted out to avoid blowing up LLVM IR. ``` Most of the others it does have are even unnecessary. Like this one. ``` /// Returns enclosing bracket ranges containing the given range or returns None if the range is not contained in a single excerpt pub fn enclosing_bracket_ranges<'a, T: ToOffset>( ``` The description is literally in the function name. Also this tweet touches on a similar issue - https://twitter.com/transmutrix/status/1750563200708309466 https://twitter.com/transmutrix/status/1750563200708309466