ID

Uploaded

Status

Description

Work Items

Action

7088
by FrankVerheyen
Oct 15, 2010
2:36 PM

Applied

Had a look at the code, and liked what I saw... almost ;-)

so I though to contribute some serious cleanup, to enhance readability and stability.

refactored, modernized and cleaned up the original code :

- got rid of (slow) ArrayList -> remodeled to (faster) generics List<...>, more type-safe
- fixed some encountered spelling mistakes
- renamed some methods and classes for better concistency
- replaced .Dispose() constructions to using(){} constructions where appropriate (especially pens, brushed, etc.)
- cleaner openfiledialog/savefiledialog usage

there's still heaps of things that could be cleaner or fixed, but hey, it's a start!

note:
Personally, I don't like non-English comments, methodnames, etc. because not everyone speaks Italian (which I guess is what the original code is in (?))...
so I replaced some of that where I could understand the meaning, without breaking down the original work or offending the writer I hope...
of course, because his work is much appreciated!

Submitting a packaged zip file with the updated code. just for fun.

Enjoy!
Frank Verheyen
Frank


Applied Oct 25, 2010: checked into svn...

Download

View All
  • 1-1 of 1 Patches
    • Previous
    • 1
    • Next
    • Showing
    • All
    • Patches