Angel Velásquez wrote:
Oh hi Michael, good, but, i have to be annoying again (and sorry) but what about with the double and simple quotes subject? i know that the file was originally written bad, but try at least to don't copy and paste things like: print "</th>"; print "</tr>"; Remember, double quotes and simple quotes are important even if you think that 'both works'. I tried to said you this in the other mail, and i thought that you understood (i hope this time you will), thanks anyway, but please try to clean the code (or maybe i say optimize it?) before submit a patch, isn't the idea to patch and re-patch with bad-written-code. Thanks!
Here's another attempt (patch attached to the mail this time), I hope I got it right this time :). Now I've change the quoting style like you said for the changed line as well as the surrounding html block. In one case I've also fixed the indentation (to use tabs) as it seemed the right step to me (that still leaves enough bad quoting to fix, but I was asked to also keep the diffs short). Kind Regards, Michael -- Michael Klier