PSeitz

Results 106 comments of PSeitz

Now you mention it, I also saw `some` used like this. I think it's bad style, because it's hard to understand but could be optionally accepted. The main difference between...

Yes, since this is type-aware it should not be mixed with #1314, but be a seperate rule. How should expensive functions handled? Flagged as expensive or moved to a seperate...

The bad compile times come from `pest_derive`, when I extract the generated code with cargo expand and remove the `pest_derive` dependency, the compile times go down from 15.3s to 3.2s...

I tried to use `cargo run --bin rustfmt -- tests/source/issue-5260.rs`, but this doesn't seem to work since the formatter seems to run with different settings: `error[internal]: line formatted, but exceeded...

> Granted if someone is unfamiliar with the test suite they might not be aware that we're parsing the //rustfmt comments for configuration values, and that might lead to confusion...

The precompiled output has sourcemaps generated. I tried it with inline and normal. Debugging works now actually better than with ts-jest, after I added an additional entry in moduleNameMapper from...

Can you convert the label to feature request? What would be the steps to achieve this? Maybe I can help

> The query parser identify the string to search for correctly. The default tokenizer splits it into several tokens (`[cherry, py]`) which triggers the phrase query. > > Probably the...

Async support has been requested for lz4_flex https://github.com/PSeitz/lz4_flex/issues/15. It's not a http compression encoding though.

llvm-mca integration is now supported by `cargo-show-asm` https://github.com/pacak/cargo-show-asm/pull/126