[ruby-core:83642] Fwd: [ruby-cvs:67787] svn:r60604 (trunk): * remove trailing spaces, append newline at EOF.

From: Martin J. Dürst <duerst@...>
Date: 2017-11-02 02:09:12 UTC
List: ruby-core #83642
Regarding the commit below, two comments:

- For some test files, it may be dangerous to always automatically
   append a newline
   (e.g., there might be a test to see whether input without a
    newline at the end is handled, or there might be a protocol
    that requires that there be no newline at the end,...)

- If for consistency, we want every file (and not just every
   C file, where we have a clear reason because some compilers
   throw a warning otherwise) to end with a newline, we should
   push that change upstream.

Similar considerations apply to spaces at the end of a line.

Regards,   Martin.


-------- Forwarded Message --------
Subject: [ruby-cvs:67787] svn:r60604 (trunk): * remove trailing spaces, 
append newline at EOF.
Date: Thu, 2 Nov 2017 08:30:14 +0900
From: svn@ruby-lang.org
To: ruby-cvs@ruby-lang.org

svn	2017-11-02 08:30:00 +0900 (Thu, 02 Nov 2017)

   New Revision: 60604

   https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=60604

   Log:
     * remove trailing spaces, append newline at EOF.

   Modified files:
 
trunk/spec/bundler/support/artifice/vcr_cassettes/realworld/api.rubygems.org/api/v1/dependencies/HEAD/request
 
trunk/spec/bundler/support/artifice/vcr_cassettes/realworld/api.rubygems.org/api/v1/dependencies-gems=bundler/GET/request
 
trunk/spec/bundler/support/artifice/vcr_cassettes/realworld/api.rubygems.org/api/v1/dependencies-gems=bundler/GET/response
 
trunk/spec/bundler/support/artifice/vcr_cassettes/realworld/api.rubygems.org/gems/bundler-1.12.3.gem/GET/request
 
trunk/spec/bundler/support/artifice/vcr_cassettes/realworld/api.rubygems.org/gems/bundler-1.12.3.gem/GET/response
 
trunk/spec/bundler/support/artifice/vcr_cassettes/realworld/api.rubygems.org/quick/Marshal.4.8/bundler-1.12.3.gemspec.rz/GET/request
 
trunk/spec/bundler/support/artifice/vcr_cassettes/realworld/api.rubygems.org/quick/Marshal.4.8/bundler-1.12.3.gemspec.rz/GET/response

[snip]

Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>

In This Thread

Prev Next