Hi, On 5 November 2012 08:39, Chris Angelico <ros...@gmail.com> wrote: > On Sat, Nov 3, 2012 at 9:15 AM, Joe Van Dyk <j...@tanga.com> wrote: > Point of random curiosity: The commit mentioned adds the following line: > > if (rinfo->reloptions && strlen(rinfo->reloptions) > 0) > > Is there a reason this isn't done as: > > if (rinfo->reloptions && *rinfo->reloptions) > > ? It seems like overkill to ascertain the string length just to find > out if the first character is the null terminator.
My guess is to be multibyte encoding safe: UTF-16 or similar. -- Ondrej Ivanic (ondrej.iva...@gmail.com) -- Sent via pgsql-general mailing list (pgsql-general@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-general
