Code cleanup

Code cleanup, removed a bunch of parser warnings and some actual undefined variables. Even though some might find this supurflous, I think it's good to keep the core code clean so we can spot those variables that are really undefined in the future.

These 4 are still questionable. I need some input on these:

The local variable $edit may not have been initialized	user.module	Drupal HEAD (Core)/modules/user	line 1161
The local variable $form may not have been initialized	search.module	Drupal HEAD (Core)/modules/search	line 245

The 2 below are still remaining, but I can see they should get set in the while. But the first time it runs through, they are undefined since $tag is FALSE.

The local variable $linknid may not have been initialized	search.module	Drupal HEAD (Core)/modules/search	line 574
The local variable $linktitle may not have been initialized	search.module	Drupal HEAD (Core)/modules/search	line 558

It's not a bad idea to flush these out, and real easy with PHPEclipse. I've caught some worthy mini-bugs just by loading up the project in Eclipse, might be a better solution than vi. :P