pfalstad / circuitjs1

Electronic Circuit Simulator in the Browser
GNU General Public License v2.0
1.65k stars 280 forks source link

Remove a bunch of warnings #25

Closed Mervill closed 1 year ago

Mervill commented 1 year ago

This PR removes a bunch of low hanging warnings, essentially the ones that could be auto-fixed without extra work. Reduces the overall warning count from 250 down to 153.

One thing that's missing from this PR is addressing the warnings about CirSim.LS(). I want to tackle these warnings with a separate PR that moves a bunch of of Locale related code to its own file.

pfalstad commented 1 year ago

Thanks!