Reorganize the code into separate files

[dabit]

While trying to figure out how the gem internals work I noticed that all the code was in a single file.

I reorganized everything into separate files and got rid of Jeweler now using the more up-to-date Bundler for releasing.

No code was changed, except for the definition of Gnuplot::Plot::QUOTED.

Used the current test suite to make sure everything is working as expected.

Later on it might be a good idea to separate the test files too.

[rdp]

Ok will merge in a week or 2 if there are no objections raised, thanks!

On Mon, Aug 12, 2013 at 4:31 PM, David Padilla [email protected]:

While trying to figure out how the gem internals work I noticed that all
the code was in a single file.

I reorganized everything into separate files and got rid of Jeweler now
using the more up-to-date Bundler for releasing.

No code was changed, except for the definition of Gnuplot::Plot::QUOTED.

Used the current test suite to make sure everything is working as expected.

Later on it might be a good idea to separate the test files too.

You can merge this Pull Request by running

git pull https://github.com/dabit/ruby_gnuplot reorganize

Or view, comment on, or merge it at:

#24
Commit Summary

• Get rid of Jeweler
• Extract Gnuplot::Plot to its own file
• Extract Gnuplot::Splot to separate file
• Extract Dataset, Array and Matrix to their own files
• Reindent and clean the code
• Get rid of RbConfig warning while testing

File Changes

• A Gemfilehttps://github.com/Reorganize the code into separate files #24/files#diff-0(3)
• A Gemfile.lockhttps://github.com/Reorganize the code into separate files #24/files#diff-1(16)
• M Rakefilehttps://github.com/Reorganize the code into separate files #24/files#diff-2(13)
• A gnuplot.gemspechttps://github.com/Reorganize the code into separate files #24/files#diff-3(14)
• M lib/gnuplot.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-4(342)
• A lib/gnuplot/array.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-5(41)
• A lib/gnuplot/dataset.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-6(78)
• A lib/gnuplot/matrix.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-7(23)
• A lib/gnuplot/plot.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-8(154)
• A lib/gnuplot/splot.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-9(19)
• A lib/gnuplot/version.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-10(3)
• M test/test_gnuplot.rbhttps://github.com/Reorganize the code into separate files #24/files#diff-11(36)

Patch Links:

• https://github.com/rdp/ruby_gnuplot/pull/24.patch
• https://github.com/rdp/ruby_gnuplot/pull/24.diff

[deric]

+1 for code reorganization

[rdp]

Shoot it has conflicts now, any chance of a rebase for me? :)