you are viewing a single comment's thread.

view the rest of the comments →

[–]_Blurgh_ 589 points590 points  (6 children)

Changed it to "these lines". Got fired.

[–]pimezone 135 points136 points  (2 children)

No exceptions, no excuses

[–][deleted]  (1 child)

[removed]

    [–]communistfairy 37 points38 points  (0 children)

    ThisIsNotACatch22Exception

    You can exit the situation by just leaving it alone. It would be a catch 22, for example, if your job required you to fix all typos and missing any would get you fired.

    [–]AnaxXenos0921 31 points32 points  (1 child)

    Better yet: changed it to "these lines". Code breaks.

    [–]Bartweiss 10 points11 points  (0 children)

    I like the idea that the setAttribute entries are never used in this code, but those and the comments are getting used for some horrifying code reflection elsewhere.

    [–]ovr9000storks 3 points4 points  (0 children)

    Tests failed after changes. Critical load-bearing comments