-
Notifications
You must be signed in to change notification settings - Fork 392
Closed
Labels
I: No breaking changePreviously written code will work as before, no one should note anything changing (aside the fix)Previously written code will work as before, no one should note anything changing (aside the fix)S: HighShould be handled nextShould be handled nextT: BugWrong statements in the code or documentationWrong statements in the code or documentation
Milestone
Description
The newest AppleClang (15.0.0.15000309) warns about assigning the result of a temporary expression evaluation result to a reference:
nest-simulator/models/astrocyte_lr_1994.cpp
Lines 84 to 89 in 5639572
| // shorthand for state variables | |
| const double& ip3 = y[ S::IP3 ]; | |
| // Ca_tot_ corresponds to the c_0 (total [Ca++] in terms of cytosolic vol) | |
| // in De Young & Keizer (1992) and Li & Rinzel (1994) | |
| const double& calc = std::max( 0.0, std::min( y[ S::Ca_astro ], node.P_.Ca_tot_ ) ); // keep calcium within limits | |
| const double& h_ip3r = y[ S::h_IP3R ]; |
conc should be a proper variable. The two references above and below are fine, since they point to vector elements.
Metadata
Metadata
Assignees
Labels
I: No breaking changePreviously written code will work as before, no one should note anything changing (aside the fix)Previously written code will work as before, no one should note anything changing (aside the fix)S: HighShould be handled nextShould be handled nextT: BugWrong statements in the code or documentationWrong statements in the code or documentation
Type
Projects
Status
Done