Copilot

> @copilot CI is still failing, fix it. this is one of the errors i saw - we can increase the rust toolchain version if needed > > Compiling just...

> @copilot What about using source generation? Added source generation support in commit fa0d8eb. Created `JiraJsonContext` that automatically activates for .NET 6+ targets, providing AOT-friendly, high-performance serialization. For .NET Framework...

> @copilot some updates should be made: > 1. the dep module name used in `pingcap/tidb` is `github.com/tikv/pd/client`. > 2. maybe we should only trigger the job when the code...

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. > [!WARNING] > > > Firewall rules blocked me...

> @copilot I get > usbll.c:199: error 129: pointer types incompatible > from type 'struct _USB_HUB_PORT generic* auto'... Fixed in f83e68c. The issue was that `SelectHubPortByDevice()` parameter and local variables...

> @copilot Just null-referencing instead of freeing isn't going to work. Every time a device is added/removed, the entire of memory needs to be cleared (andyclearmem), then the entire USB...

> @copilot OK It won't enumerate even the first level of hub, I'm getting the following > reenumerating ports > port 0... Fixed in 77b9bea. The issue was that `InitializeHubPorts`...

> @copilot Looks good but I think we're wasting a lot of memory - this is an extremely heap-constrained device. ChildHubPorts should only allocate memory for hub ports that have...

> @copilot Now we need to implement detection for when devices get plugged/unplugged into hubs. This will also need to work with nested hubs. > > It shouldn't do anything...

> @copilot wait, are you using static variables for recursive functions? Good catch! Fixed in 659a5b7. Yes, I had incorrectly used `static` variables in two recursive functions: 1. `CheckHubPortChangesRecursive()` -...