I am amazed.
Today, while rewriting and extending some code for the development (and subsequently, the production) environment, I was amazed at some of the code I found. In particular I found code relating to the calculate of checkpoint barcodes that did not conform to either the job specification supplied by the client or that prepared for implementation internally - At the point where the code was supposed to build a barcode for article tracking based upon the unique job number; and where a checksum was supposed to be incorporated into the barcode, a padded zero was instead inserted.
When replacing code like this, it doesn't make it hard to return an "improvement" in the time invested. Now just to make sure my code actually works *grin*
