• (nodebb)

    Stuart was also the kind of person who would suddenly go on a tear, write three thousand lines of code in an evening, and then submit an pull request. He wouldn't respond to PR comments, however, and just wait until management needed the feature merged badly enough that someone said, "just approve it so we can move on."

    So TRWTF is that nobody in a position to do so has the backbone to flatly refuse to allow Stuart's PRs to be merged unless Stuart handles the comments.

    Or maybe TRWTF is that nobody is in a position to do that.

  • Officer Johnny Holzkopf (unregistered) in reply to Steve_The_Cynic

    I wouldn't be surprised when the simple answer is: Nobody actually cares...

  • (nodebb)

    He wouldn't respond to PR comments, however, and just wait until management needed the feature merged badly enough that someone said, "just approve it so we can move on."

    This is the most obnoxious kind of developer and one that needs to be thrown out on their butts ASAP.

  • Hans (unregistered)

    So, if bit 0x1000 is set, the background must be both ON and OFF?

  • LZ79LRU (unregistered) in reply to DocMonster

    No. Under no condition should you toss individuals like this out a high rise window such that they land on their soft posterior. Always, and I mean always ensure they go down head first.

  • (nodebb) in reply to Steve_The_Cynic

    No technical management, only business management, and techs are all just the same blob: Fights. Happens not only in IT, but IT always have this to a degree.

  • (nodebb)

    "The code is confusing and unnecessary." Probably can also be said about the management.

  • (nodebb)

    I was once asked to review a 500 file PR and declined since its impossible. Later I saw all the comments were LGTM.

  • (nodebb) in reply to LZ79LRU

    If the window is high enough, it isn't important (other than the need for ensuring that others are outside the "splat zone").

  • (nodebb)

    The system I work on once had a "developer" who seemed to think that having an "isEnabled" property on a particular object was insufficient, so added an "isDisabled" property (and then failed to ensure they stayed in sync...).

    He also had a habit of writing a pretty standard conditional [ IF (condition) then (whatever) ], then, if it didn't work right, come through and add a negation [ IF not (condition) then (whatever) ]. If that still didn't work (or something else changed), he'd add another 'not', then another, and another... A few years later, another programmer found some of his code tucked away in a rarely if ever called corner, which was of the form [ IF not not not not not not (condition) then....]

  • NobodySpecific (unregistered) in reply to LZ79LRU
    Comment held for moderation.
  • John (unregistered)
    Comment held for moderation.
  • (nodebb)

    Also, why do some variables have Hungarian notation (iDisplayFlags, bForceBackgroundOn, bForceBackgroundOff) and some don't (useBackground)?

  • (nodebb)

    I almost enjoy that I can't tell if this was written in "object-oriented" C99 or a language from the current millenium.

  • Stuart (unregistered)

    I just want to make it clear that this WAS NOT ME. It was my evil twin. I denounce his actions as being against the true spirit of Stuartness.

  • (nodebb) in reply to Maia-Everett

    If you use Hungarian with C#, you don't follow the official style guide that is around for over two decades and was published before .net 1.0 went out of Beta. So the question is not why partially using Hungarian, but why did someone not follow the official style guide that literally everyone else is using. It just ends up being a mess.

  • LZ79LRU (unregistered) in reply to dkf
    Comment held for moderation.

Leave a comment on “On Deep Background”

Log In or post as a guest

Replying to comment #:

Log Out

« Return to Article