Notice: On April 23, 2014, Statalist moved from an email list to a forum, based at statalist.org.
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: st: Minor bug in -labvalclone- by NJC
From
Nick Cox <[email protected]>
To
"[email protected]" <[email protected]>
Subject
Re: st: Minor bug in -labvalclone- by NJC
Date
Fri, 22 Nov 2013 01:04:49 +0000
Thanks to Sergiy for the public heads-up.
I will get it fixed.
Nick
[email protected]
On 22 November 2013 00:45, Sergiy Radyakin <[email protected]> wrote:
> Bug in -labvalclone- by NJC
> http://fmwww.bc.edu/repec/bocode/l/labvalclone.ado
>
> ===Example:
> clear
> label define a 1 "US" 2 "UK" 3 "UA"
> labvalclone a aa
>
> ===Output:
> unrecognized command: laabel
> r(199);
>
> ===Why does it happen:
> Because replacing blindly code is not safe. Nick is trying to check
> for this problem by having a safety check of the old argument earlier
> in the program:
> if "`old'" == "label" | "`old'" == "define"
> but later is doing a replacement of a SUBSTRING, not a WORD-by-word replacement.
>
> ===Suggested fix:
> Simply flank both args (old and new) with spaces:
> local line: subinstr local line " `old' " " `new' "
>
> Best,
> Sergiy Radyakin
> *
> * For searches and help try:
> * http://www.stata.com/help.cgi?search
> * http://www.stata.com/support/faqs/resources/statalist-faq/
> * http://www.ats.ucla.edu/stat/stata/
*
* For searches and help try:
* http://www.stata.com/help.cgi?search
* http://www.stata.com/support/faqs/resources/statalist-faq/
* http://www.ats.ucla.edu/stat/stata/