Re: [Jsynthlib-devel] Fwd: [Jsynthlib-cvs] SF.net SVN: jsynthlib:[1104] trunk/JSynthLib/core/Actions.java

[Joe E. <jo...@em...>]

On 09/04/2011 02:08 PM, frankster wrote:
> On 03/09/2011 15:57, Joe Emenaker wrote:
>> On 9/3/2011 6:59 AM, frankster wrote:
>>> * which files are you planning to include in the refactor so I can avoid
>>> changing those files?
>> I've tracked down which revision I started the refactor on (it's from
>> back in 2006...  yeah, really), so now I'm doing a diff on all of the
>> files to see what I've changed.
> Must be some kind of world record!

For procrastination? :-O

Okay, here's a sort-of list. What I did was run diff on every file from 
revision 1051 to compare it to my refactoring code and then I ran each 
diff into a line-counter so that I could see what files have changed the 
most...

308 JSynthLib.1051/JSynthLib/core/MidiUtil.java
224 JSynthLib.1051/JSynthLib/core/Actions.java
74 JSynthLib.1051/JSynthLib/core/SysexGetDialog.java
74 JSynthLib.1051/JSynthLib/core/DriverUtil.java
60 JSynthLib.1051/JSynthLib/core/AbstractLibraryFrame.java
53 JSynthLib.1051/JSynthLib/midiprovider/OSX/CAProvider/CAMIDISource.java
38 JSynthLib.1051/JSynthLib/core/Scene.java
36 JSynthLib.1051/JSynthLib/core/DevDrvPatchSelector.java
32 JSynthLib.1051/JSynthLib/core/BankEditorFrame.java
28 JSynthLib.1051/JSynthLib/core/PatchEdit.java
23 JSynthLib.1051/JSynthLib/core/PatchEditorFrame.java
12 JSynthLib.1051/JSynthLib/org/jsynthlib/editorbuilder/EditorBuilder.java
8 JSynthLib.1051/JSynthLib/JSynthLib.java
8 JSynthLib.1051/JSynthLib/core/ErrorMsg.java

I'm a little surprised that so few lines have been altered. I'm also 
surprised that MidiUtil is the top of the list, but I just looked at my 
copy and I now remember that I've made the whole thing its own process 
thread which dispatches MIDI messages to the appropriate driver, etc. 
After that, Actions is the class that has been modified pretty heavily.

- Joe