I would add that stored procedure names should not start with sp_ because SQL Server will search the MASTER database for the stored procedure first and then the user database. Although the performance hit is minor it is still a hit.

For similar reasons always prefix object names with their owner i.e. dbo.tblMyTable or dbo.vwMyView.

Always comment your code unless you want to be shackled to it for life.

Always carry out a QA stage on code.

If at all possible, make developers use stored procedures, not dynamic SQL.

Anyone who uses ADO should use the command object to call the stored procedure rather than connection.open "SELECT * FROM...etc"

========================

He was not wholly unware of the potential lack of insignificance.