Steven Schveighoffer

> The key parameter should not be passed to foreach loops by ref, so your hopefully only mutating your copy of it. I'm guessing you misunderstood "value of key" like...

@quickfur that's all good, but modifying a value that's in an AA is expected to do nothing to the structure. It's perfectly acceptable, and I would say many people rely...

FYI, I think all the deprecated enums should be turned off by this version, but I didn't take the time to find them all. If this technique looks good, I...

I think just `__traits(isDeprecated)` is the only way to use a symbol without triggering the deprecation warning. There really isn't a way to have generic code access deprecated members without...

I'll close this for now. If the changes you are making fix my issue then I'll close that too. I *think* it will work, but I'm not sure if I'm...

This is because it treats a `null` as `not present` in the case of `@optional` being attached: https://github.com/vibe-d/vibe.d/blob/7161dc18c62120d555b099de8293b32a02a490e0/data/vibe/data/serialization.d#L911 I think this is an incorrect design decision, but I'm not sure...

Thought about this, a solution could be to loop over the members twice, and add the redirects at the end. This way redirects are only chosen if the main routes...

looking at the usage (figure out which imports need to happen to include these symbols), I think this change is correct in terms of what this thing is meant to...

TypeTuple not defined. Plus it's deprecated, this example should be updated to use `std.meta.AliasSeq` (with the appropriate import)

Note, this doesn't change any code, so targeting master/stable is irrelevant. Verify the result in the doc generator preview before merging.