Adjacent strings auto-concatenation not implemented in VM

[sethladd]

The removal of + from String has necessitated a way to handle long, multi-line strings. The proposed solution:

var superLongString = 'this is a crazy long string and yet I cannot stop typing it'
                                 'even after I go to another line the typing, it just does not stop';

This is not yet implemented in VM.

[kasperl]

I think this is already implemented. Matthias?

---

Set owner to @mhausner.
Added Started label.

[DartBot]

This comment was originally written by @mhausner

---

Yes, this is done.

---

Added Fixed label.

[DartBot]

This comment was originally written by rodion...@unipro.ru

---

It seems that multi-line strings are concatenated as well now, although spec doesn't mention they should be. (10.5: Adjacent single line strings are implicitly concatenated to form a single string literal.)
Which one is in the wrong: VM or spec?

Example:
  var s = 'a'
          "b"
          '''c'''
          """d"""
          """
e"""
          '''
f''';
  Expect.equals('abcdef', s);

[DartBot]

This comment was originally written by @mhausner

---

Gilad, can you please clarify the spec if necessary?

---

Set owner to @gbracha.
Removed Area-VM label.
Added Area-Language, Triaged labels.

[gbracha]

The grammar allows for adjacent strings to be concatenated in both the single line and multiline spec. The text did only discuss this for single line strings; I am correcting this in version 0.11.

---

Added this to the M1 milestone.
Added Accepted label.

[gbracha]

Fixed in 0.11 draft

---

Added Done label.

[DartBot]

This comment was originally written by @tomyeh

---

This feature seems to be error prone. For example, when instantiating a list of strings, two consecutive strings usually imply the missing of a comma rather than concatenation:

var list = ["apple" "orange"];

With this feature, it is hard to spot an error like this. I'd suggest to simply signal an error rather than concatenate them together. After all, multiline strings are powerful enough to handle it.

[sethladd]

Hi Tom, This might be a good feature request for Dart Editor. The editor could issue a warning here (or a to-be-defined "head's up" feedback) about this case. Please open a new feature request if you agree. Thanks for the feedback!

[DartBot]

This comment was originally written by @tomyeh

---

Could do but, IMO, it is more about the language spec. If Dart team still thinks the value greater than the cost, we have to get used it. No language is perfect. However, I think it is an over-kill feature. IMO, with multi-line string and ${}, I don't see there is a need for this feature.

BTW, I don't use Dart Editor. It is great but not fast enough comparing to Sublime or like.