[
https://issues.apache.org/jira/browse/SYSTEMML-1258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15867223#comment-15867223
]
Matthias Boehm commented on SYSTEMML-1258:
------------------------------------------
Yes, this is because we literally map the append to cbind even before
validation.
The only difference between the string append and string concatenation (+
operator) is that the string append adds a newline. However, since we meanwhile
support escape characters, I would actually be in favor of completely removing
this string append. It would require to change a number of scripts - but I
think it would actually look much cleaner.
> cbind appends strings
> ---------------------
>
> Key: SYSTEMML-1258
> URL: https://issues.apache.org/jira/browse/SYSTEMML-1258
> Project: SystemML
> Issue Type: Bug
> Components: Parser, Runtime
> Reporter: Deron Eriksson
>
> The append() method is documented to allow (1) cbind() (column append)
> functionality for matrices and (2) allows strings to be appended to other
> strings.
> However, cbind() also seems to allow strings to be appended to other strings.
> Since cbind() is a matrix function, this seems incorrect.
> {code}
> makes = "makes";
> makes_sense = append(makes, "sense");
> print(makes_sense);
> doesnt_make = "doesn't make";
> doesnt_make_sense = cbind(doesnt_make, "sense");
> print(doesnt_make_sense);
> {code}
> output:
> {code}
> makes
> sense
> doesn't make
> sense
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
