nathaniel-daniel
nathaniel-daniel
Wouldn't enabling this feature once for this library enable it for everything though? It seems worse to force people to go to each dependency and find the feature flag to...
True, but enabling the parallel feature will also pull in a new dependency. If this dependency is unwanted, it becomes much harder to remove it since default features are difficult...
Including a build dependency when it isn't needed will increase the build time, which will slow down the overall build. Some builds will also not benefit from a parallel build....
It is possible to enable features for a `cargo install` build with the `--features` flag. One could add a feature to their binary to easily allow users to make their...
Maybe it would make sense to add a shortcut method to `ElementRef` for such a common usecase?
> A macro is nicer in this case because it's easier to return compile errors which reduce the need for run-time debugging. How would a macro return a compile-time error?...
Sorry, I meant how would a macro know if a selector was invalid?
The function passed to `config_log` must also not call sqlite in any way, and I don't think its possible to prove that automatically.
I think this is due to the line [here](https://github.com/m4b/goblin/blob/0.5.4/src/archive/mod.rs#L163) directly indexing into the name str without previously validating the byte indexes. In some cases, this could split a multibyte codepoint,...
I think the issue is https://github.com/rust-lang/rust/issues/98947?