search

lcpp-org / crane

A MOOSE application dedicated to general Chemical ReAction NEtworks for plasma chemistry and thermochemistry problems.
https://crane-plasma-chemistry.readthedocs.io/
GNU Lesser General Public License v2.1
21 stars 20 forks source link

Add fixups for -Werror with gcc #27

Closed lindsayad closed 5 years ago

lindsayad commented 5 years ago

These variables were all local variables that were unused. In the _energy_change case you actually had the dangerous behavior that you were modifying a local when you probably thought you were operating on the class member variable.

keniley1 commented 5 years ago

I know there are several instances where I made local variables that were used at the time, but future updates left them behind without removing them. Thanks for this!