therneau / survival

Survival package for R
381 stars 104 forks source link

Use inplace decrement operator #221

Closed MichaelChirico closed 1 year ago

MichaelChirico commented 1 year ago

As flagged by our compiler.

I don't know the code well, so I am not 100% sure this is correct, but glancing around I see += / -= used for these *temp objects, so I think it is (the alternative is ntemp = -wt[p1];, which is what the current code is doing, but with clearer spacing).

therneau commented 1 year ago

Your compiler is right, it is a typo. Now fixed in my master, will soon be pushed