Anything involving any level of technical difficulty is wrong. And should be done another way, even if it is not optimal and only slightly more readable.

Pointers are bad no matter what.

No locks.

Only ifs, no else ifs or else's, even when it makes sense.

Like I understand readability and the importance of design in simplifying solutions, and I've learned a lot from them, and they are great testers, but sometimes things just feel like micromanaging. I'm grateful to be learning from them but I feel like anything that isn't something they want to do is wrong, so I'm not sure how much I'm learning anymore vs how much they just want things done their way.