My task is to remove duplicate and unused files from a production CI application. I've identified errors but not certain I can move a entire file to junk or just a portion. View files have been reduced by 80% and Models have been mostly inactive. Currently the focus is on approximately 100 controllers (80 have been named to indicate they are a prior version) with average 1500 lines and 4 library(s) of 5,000+ lines.
This is line 1 for a number of php files in the controller folder.
<? if (!defined('BASEPATH')) exit('No direct script access allowed');
should it be:
<?php if (!defined('BASEPATH')) exit('No direct script access allowed');
Would this be enough to put it in junk or is there something else that should be maintained as a priority?