CBielstein / APRSsharp

APRS# - Modern APRS software for the amateur radio community
MIT License
12 stars 5 forks source link

Consolidate FindCorrectYear, FindCorrectDayMonthAndYear, FindCorrectYearAndMonth #61

Open CBielstein opened 3 years ago

CBielstein commented 3 years ago

Timestamp has three methods that appear to do very similar work. Can they be consolidated somehow? Perhaps using optional parameters, etc. Those methods are FindCorrectYear, FindCorrectDayMonthAndYear, and FindCorrectYearAndMonth.

FindCorrectYear also does some weird...while(true) catch loop. Investigate if that can be replaced with a cleaner approach.