Project

General

Profile

Actions

Feature #5337

closed

Use outbufs in FileUtils.compare_stream

Added by Glass_saga (Masaki Matsushita) over 12 years ago. Updated over 12 years ago.

Status:
Closed
Assignee:
-
Target version:
[ruby-core:39622]

Description

=begin
I propose using outbufs in FileUtils.compare_stream.
It will increase performance.

I executed the following code to performance test.

require 'fileutils'
require "stringio"

str = "hoge" * 100000000

a = StringIO.new(str)
b = StringIO.new(str)

FileUtils.compare_stream(a, b)

The following are results of time command.

present compare_stream:

1.27s user 0.69s system 96% cpu 2.036 total
1.38s user 0.69s system 96% cpu 2.152 total
1.12s user 0.24s system 98% cpu 1.379 total

patched compare_stream:

0.70s user 0.43s system 97% cpu 1.158 total
0.72s user 0.27s system 98% cpu 1.010 total
0.62s user 0.28s system 98% cpu 0.915 total

Patched ruby passes tests in test/fileutils.

=end


Files

patch.diff (646 Bytes) patch.diff Glass_saga (Masaki Matsushita), 09/19/2011 06:01 PM
Actions #1

Updated by ayumin (Ayumu AIZAWA) over 12 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r33297.
Masaki, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/fileutils.rb (module FileUtils): improve performance of
    FileUtils.compare_stream. a patch by Masaki Matsushita.
    [Feature #5337] [ruby-core:39622]
Actions

Also available in: Atom PDF

Like0
Like0