From: "usa (Usaku NAKAMURA)" <usa@...>
Date: 2012-10-25T13:41:52+09:00
Subject: [ruby-core:48221] [ruby-trunk - Feature #6265][Assigned] Remove 'useless' 'concatenation' syntax


Issue #6265 has been updated by usa (Usaku NAKAMURA).

Status changed from Closed to Assigned
Assignee changed from mame (Yusuke Endoh) to drbrain (Eric Hodel)

Now I've committed mame-san's patch (I was asked by him to commit instead of him).

But this change introduced some failures to make test-all, because rubygems
uses string concatenation at lib/rubygems.rb:1246-1249.
Eric, could you change these lines?
----------------------------------------
Feature #6265: Remove 'useless' 'concatenation' syntax
https://bugs.ruby-lang.org/issues/6265#change-31485

Author: rosenfeld (Rodrigo Rosenfeld Rosas)
Status: Assigned
Priority: Normal
Assignee: drbrain (Eric Hodel)
Category: core
Target version: 2.0.0


What is wrong with this code:

some_method 'argument1', 'argument2' 'argument3'

Yes, the missing colon, but it is not always easy to notice that...

What is this ('concatenation' 'syntax') useful for?

Why writing ('some ' 'concatenation') instead of 'some concatenation'?

A missing colon between string arguments can lead to some bugs that may be hard to find, specially if the arguments are optional.

And I can't see any useful case where this allowed syntax for concatenation would help.


-- 
http://bugs.ruby-lang.org/