Changed a few "strings" to 'strings', for better optimisation. Will test soonishly.

CommentFileSizeAuthor
#1 userplus.module.double-to-single-quote_0.patch4.71 KBFreso
userplus.module.double-to-single-quote.patch3.06 KBFreso

Comments

Freso’s picture

Freso commented
Title: Changed t("string") to t('string') » Changed "strings" to 'strings'
StatusFileSize
new userplus.module.double-to-single-quote_0.patch4.71 KB

Found some SQL in double quotes as well, so I included them as well. Still not tested though.

heine’s picture

heine commented

Before going on a double quote killing spree, it may be worthwhile to do a little benchmarking :-).

Freso’s picture

Freso commented

True. But, either way, I doubt t('') is faster than '' - though that is actually another of the other issues I filed, now that I think about it.

Anyway, this is that of the issues of which I care the least. I'll much rather have the four other issues fixed than this one. But if this is of any help, then it's all good, if it isn't, well, I was messing around with the code anyway. :P

Freso’s picture

Freso commented
Status: Needs review » Fixed

Fixed: http://drupal.org/cvs?commit=33997

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)