-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
refactorimprove code qualityimprove code quality
Description
Background: dirhash.py contains lots of different functions and no classes.
Problem: it's getting crowded
Solution: Separation of concerns by splitting the functions in to classes:
- one class for low-level hashing stuff: This are pretty much all private function, and a couple of public ones
- one class for high-level file-handling-stuff and wrapper for low-level stuff.
todo:
- choose name for the 2 classes. some first ideas:
- hash_driver and handler/frontend
- dirhash and handler
- move functions, and add private members, class-variables, constructors, etc. accordingly
- get rid of most global vars (turn them into private members in their respective class)
- update dirhash_test.py
- update run.sh
Metadata
Metadata
Assignees
Labels
refactorimprove code qualityimprove code quality