Here are some words Linus used to describe the submitted code: crazy, stupid, garbage, needs to get bent, incomprehensible.
Would you like to work at a place where these words are used to give feedback? I sure wouldn’t. But maybe I’m a snowflake.
Would you like to work at a place where these words are used to give feedback? I sure wouldn’t. But maybe I’m a snowflake.
So the commentary was on more than just the code it was on the entire situation. It was concluded by saying they could do this but not in generic headers and they could try the PR again early in the next release.
That language was not meant to be inflammatory but stern and the door was left wide open for future submissions.
Would I mind this in my workplace? Probably not. Particularly if my boss was a well known programmer with decades of experience and inside knowledge to share with me.
I get the sense that he’s more upset by the timing, and whilst unimpressed with the code, he’s reacting because of the timing. Hard to tell though because of the way that email is written.
Context matters here.
That way allows for tyrants.
I’m not saying it’s OK for everyone. If folks are unable to extract the same value that’s fine, find advice from someone else.