Ticket #91 (closed task: fixed)

Opened 4 years ago

Last modified 4 years ago

Code Cleanup

Reported by: Joel Owned by: Joel
Priority: trivial Milestone: Eraser 6.0
Component: Core Version:
Keywords: Cc:
Processor Architecture: Blocked By:
Blocking: Operating System:

Description

While going through the code I see a big difference between newly written code and old code (written at the beginning). These have to be fixed:

  • No public variables. Properties are preferred
  • No internal variables and Public get properties. Use internal set instead
  • Various documentation inconsistencies

Blocking

IdSummaryMilestone
#91└ Code CleanupEraser 6.0

Blocked by

IdSummaryMilestone
#91└ Code CleanupEraser 6.0

Change History

comment:1 Changed 4 years ago by Joel

  • Status changed from new to accepted

comment:2 Changed 4 years ago by Joel

  • Priority changed from major to trivial

comment:3 Changed 4 years ago by Joel

  • Status changed from accepted to closed
  • Resolution set to fixed

I think this is done this week from r898 to about r960

Note: See TracTickets for help on using tickets.