should we use BaseTool.SUCCESS or just mark return codes with comments?