search

ROMaster2 / LiveSplit.VideoAutoSplit

Video Auto Splitter for LiveSplit
MIT License
37 stars 14 forks source link

attempt at cleanup #27

Closed kenman345 closed 5 years ago

kenman345 commented 5 years ago

Small little cleanup

kenman345 commented 5 years ago

Feel free to squash these commits, it got a little messy with the capitalizations

kenman345 commented 5 years ago

@ROMaster2 I made a change in two constructors so we dont have redundant initialization of objects needing to be updated if things change or are removed.

I was also wondering about a lot of the Models/Features classes. It would seem that we have some with a public constructor and an empty internal constructor, but on others both the constructors are set to internal. Which do you think is right and would it be okay for me to add that to this PR?