[wp-hackers] Re: [wp-svn] [4656] trunk: new function for escaping within attributes: attribute_escape()

[Les Bessant]

Speaking as someone who just looks at small bits of code for very minor
tweaks, descriptive function names (and variable names) are always a Good
Thing. 


Les Bessant les at lcb.me.uk
Losing it[1] - http://lcb.me.uk
My flickr pictures - http://flickr.com/photos/lesbessant/


> -----Original Message-----
> From: wp-hackers-bounces at lists.automattic.com [mailto:wp-hackers-
> bounces at lists.automattic.com] On Behalf Of Viper007Bond
> Sent: 21 December 2006 13:34
> To: wp-hackers at lists.automattic.com
> Subject: Re: [wp-hackers] Re: [wp-svn] [4656] trunk: new function for
escaping
> within attributes: attribute_escape()
> 
> Is it really that big of a deal having a few extra characters? I'd much
> rather have the function name be clear as to what it does rather than have
> it be as short as possible, especially since there are LOADS of other
places
> where we could trim down filesize if it was actually an issue (and I
> personally don't think it is).
> 
> On 12/21/06, Nikolay Bachiyski <nbachiyski at developer.bg> wrote:
> >
> > 2006/12/21, m at wordpress.org <m at wordpress.org>:
> > >
> > >
> > >  Revision 4656 Author markjaquith Date 2006-12-21 10:10:04 +0000 (Thu,
> > 21
> > > Dec 2006)
> > > Log Message new function for escaping within attributes:
> > attribute_escape()
> > >
> >
> > Couldn't the funcion name be somehow... shorter?
> >
> > Happy hacking,
> > Nikolay
> > _______________________________________________
> > wp-hackers mailing list
> > wp-hackers at lists.automattic.com
> > http://lists.automattic.com/mailman/listinfo/wp-hackers
> >
> 
> 
> 
> --
> Viper007Bond | http://www.viper007bond.com/
> _______________________________________________
> wp-hackers mailing list
> wp-hackers at lists.automattic.com
> http://lists.automattic.com/mailman/listinfo/wp-hackers