I agree the "show" output doesn't look right.
[BTW, after the change is committed the "show" output looks normal.]
Though, from your fix it looks like the "for ..." loop is not
needed at all, and it should be replaced with:
result = "\"" + value + "\"";
return result;
Please submit a Bugzilla entry so the issue can be tracked and
the fix can be tested properly.
Thanks,
Pavlin
杨小帅 <[EMAIL PROTECTED]> wrote:
> when you entry a "xxx xxxx" value into txt node. The xorpsh will
> pass it. But when you use show -all in CLI, the information in your
> screen is wrong. Apparently, text should not be "\"abc cba\"".
>
> exp:
> [EMAIL PROTECTED] set vlans interface 312 text "abc cba"
> [EMAIL PROTECTED] show
> vlans {
> interface 312 {
> > text: "\"abc cba\""
> }
> }
>
> BUG FIXED:
> string
> ConfigTreeNode::quoted_value(const string& value) const
> {
> string result;
> ...
> if (x == '"')
> > escaped += "\\\""; //should be escaped += "";
> else
> escaped += x;
> ...
> return result;
> }
>
>
> B.R.
> Xiaoshuai Yang
>
> _______________________________________________
> Xorp-hackers mailing list
> [email protected]
> http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/xorp-hackers
_______________________________________________
Xorp-hackers mailing list
[email protected]
http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/xorp-hackers
