3 x undefined variable/index errors + coder style cleanup

Thanks. This could take a short while to review, but I'll make sure to do it before the patch gets stale. We can't apply to just one branch of Webform however. Could you resolve the conflicts when this patch is applied to Drupal 5?

In order to maintain both versions of the module, they need to be as consistent as possible in every way (*especially* spacing and code style consistencies). Otherwise every patch that comes up takes forever to apply to both versions.

Okay... finally committed. Thanks for the general code style fixes.

However. Please do not EVER submit another issue to any of my project that deal with switching double quotes to single quotes. I don't care. Regarding the use of drupal_* functions such as drupal_strtolower() or drupal_substr(), these are completely unnecessary in situations where we already know what the string values are (since they're in code originally). I don't have statistics, but I'd bet a single call to drupal_substr() negates a million double quotes. Have you looked at the code for that function? I also cleaned up a dozen misuses of update_sql(), which cannot take parameters so this patch would have broken all update paths.

I'm thankful for the patch, but really this could have totaled 300 lines instead of 3000. I mostly committed it because I know it was a lot of work for you to do both of these patches. So thanks again. Please chill out.