gosumemory icon indicating copy to clipboard operation
gosumemory copied to clipboard

Complete rewrite

Open l3lackShark opened this issue 4 years ago • 3 comments

Original architectural structure decisions were made back in the day when I was just starting coding and so it really is a mess. Was honestly surprised how stable it became after so many crutches and polishing attempts.

Current flaws:

  • Blocking loops on each goroutine instead of a sequential operations/channels
  • Asynchronous websocket stream as a result of ^
  • Cgo calls mix within go calls/multiple instances of pp calculators
  • Bad error handling (non-existent in some cases)
  • .ini config was a mistake after all, we probably should move to something like json/yaml
  • Mix of {log/pp/fmt}.Println calls for no reason
  • Global variables mess across packages
  • Non-Existent debug logging (besides mem package)

To combat that until it's too late I made a decision to rewrite most of the project.

The progress will be updated below:

  • [ ] replace a mess that's also known as a pp package with https://github.com/l3lackShark/cgoppai
  • [ ] rewrite memory package to the point that it's consistent with the way we read memory and send payload to web
  • [ ] GUI?

l3lackShark avatar Mar 31 '21 19:03 l3lackShark