Seiichi Uchida

FYI: `Entry` for `SkipMap` and `SkipSet` implements `Drop`, which in turn calls `release_with_pin` so these two types don't suffer from memory leak. I don't necessarily think this is a bug...

I have observed that nested parens will significantly slow down the parser. ```glsl void main () { (((((((1.0))))))); } ``` Parsing the above file with the release build takes about...

Thank you for pointing me in the right direction for debugging. I added a timeout, and it turned out that many tests were failing due to the "too many open...

FYI: lstopo 

Here is the output: ``` ❯ cat /sys/devices/system/cpu/online 0-47 ❯ cat /sys/devices/system/node/online 0-3 ❯ for i in (seq 0 3); cat /sys/devices/system/node/node$i/cpulist; end 0-5,24-29 12-17,36-41 6-11,30-35 18-23,42-47 ```

Concerning 3., I have submitted #478 for the possible fix. Given that it's not reproducible in every environment, I imagine that this is not a priority for you, so please...

> `physical_package_id` They are all zero :( ``` ❯ for i in (seq 0 47); cat /sys/devices/system/cpu/cpu$i/topology/physical_package_id | tr \n ' '; end 0 0 0 0 0 0 0...

Yeah, I am happy to contribute, hopefully finishing by this weekend. > why the physical_package_id is zero I am by no mean expert on Linux...but looking at the documentation, [`physical_package_id`](https://www.kernel.org/doc/html/latest/admin-guide/abi-stable.html#abi-sys-devices-system-cpu-cpux-topology-physical-package-id)...

> One thing I did notice is that it doesn't format code that is hidden by prefixing it with a #. It's not terrible but it would be nice if...

Since rustfmt acts on AST, which holds little or no type information, skipping based on type can only be done heuristically. > if an alias `type Affine = Matrix3` would...