BIRT 3.7
Written by: Michael Williams
Featured Refcardz: Top Refcardz:
  1. Scrum
  2. Apache Maven 2
  3. Essential MySQL
  4. Node.js
  5. Groovy
  1. jQuery Selectors
  2. Ajax
  3. Java
  4. Spring Config.
  5. Java Concurrency

Link Details

Link 152066 thumbnail
User 111696 avatar

By bloid
via paulbiggar.blogspot.com
Published: Jan 27 2009 / 18:25

I'm inventing a new sport today, which I call "malicious code reviews". I spent a few hours reading some really very bad code, and in retaliation against its author(s), I'm going to code review it [1]. The code comes from PHP version 5.2.8, the latest stable release. This particular file is Zend/zend_operators.h [2]. You might want to open it in a new window, or in a popup, so that you can follow along.
  • 11
  • 2
  • 1534
  • 0

Comments

Add your comment
User 209464 avatar

willcode4beer replied ago:

0 votes Vote down Vote up Reply

Why not fix and submit a patch instead?

User 276794 avatar

Loïc Hoguin replied ago:

0 votes Vote down Vote up Reply

Because that would be common sense.

A 10+ year old code base contains cruft. News at 11.

Add your comment


Html tags not supported. Reply is editable for 5 minutes. Use [code lang="java|ruby|sql|css|xml"][/code] to post code snippets.

Voters For This Link (11)



Voters Against This Link (2)