|Anonymous | Login | Signup for a new account||2017-05-24 23:12 UTC|
|Main | My View | View Issues | Roadmap | Repositories|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0000636||OLV||General Features||public||2012-07-31 17:47||2012-08-22 21:16|
|Target Version||0.0.3||Fixed in Version||0.0.3|
|Summary||0000636: Pre-0.0.3 release code clean-up using mvn site.|
|Description||Make Fred happy.|
|Tags||No tags attached.|
A few points:
1) Leave whitespace/formatting changes till last.
2) Don't change optionpanev2 at all, it's going to be gone soon, waste of repo space to do so
3) Any non-cosmetic changes should be very carefully checked. Get someone else in IRC to help you test it, direct them on what to try, use temp branches if unsure.
4) Don't mess with FreeEMSBin at all
See these reports:
findbugs - case by case
checkstyle - not just style in here, last release was 383, last hash i pulled has 449, aim for under 300. do this last
javancss - review stuff with high CCN numbers
cpd - consider breaking out common chunks into sub-methods
pmd - likely mostly bogus, perhaps we can configure this to be less zealous or more specific
jdepend - review this, try to understand why it defines OLV code as bad code, refactor stuff if you feel like it
consider upping the effort and lowering the threshold on findbugs to draw out more issues.
edited on: 2012-08-13 18:52
A good bit of this has been completed.
A little bit of CCN (high complexity) was fixed.
A couple bugs found by the bug finder were fixed.
Style points went from 383 at 0.0.2 release to 488 when I started doing the cleanup and it is at 300 now.
I'm halting work on this until the rest of 0.0.3 is completed and it is time to check this stuff again.
The git hash with the changes from this update is: 2523f54ee50532f5793460b89749edeab8697ac2
Work completed in git hash: 462aecc306d6f73452c5c2487518863e5c1ed7b4
Much better code reuse for NavBarPanel.java. Also remove superfluous use of 'this'.
Checkstyle error count after last wave was 300, after doing more real work it crept up to 329 and now after this second and final wave of clean-up it is down to 260. The release of 0.0.3 is not far away now. =D
|Reviewed diff, looked good. Checked app too, looks good!|
|Copyright © 2000 - 2011 MantisBT Group|