prDiet minor tweaks

Noticed that breast redistributors dropped one of my slaves to a boobs value of 35. Have gone through the code and added limits to risky sources of breast shrinkage, to avoid breast size falling below the current flat value of 100. Have two major concerns before I un-draft:

1. Did I go too far adding the drug cessation conditions to PC breast redistributors?

2. More importantly, should this be setting Math.max(x.boobs, 100 + x.boobsImplant + x.boobsMilk) instead of Math.max(x.boobs, 100) as I currently have it? I'm pretty sure I'll need to anywhere that defines boobSize, but not sure if it's warranted elsewhere.

UPDATE: This MR now just tweaks some boobSize comparisons and spaces out a subtraction operation.

Technically, as long as it's not 0 or below, it is a valid breast value. For the most part it's considered irrelevant, but it does delay growth in rare cases, which is why it isn't just clamped to 100 during cleanup.

Drug cessation should happen in the drugExpiry() block or equivalent, something that the slave side needs to be updated to actually do.

Whoops, completely overlooked that block. Should I just discard all of these changes, then?

I think if nothing else, switching any instances of boobSize >= 100 to boobSize > 100 would at least make sense.

Either way, they're both below the point that values are directly reported, but yeah that would be fine.

Ready to go, then.

added 2 commits

• 5bc840dd - Revert "PC.boobs lower limit"
• 4bb64fff - Revert lower boobs limits

Compare with previous version

added 1 commit

• 21c4e55c - Minor prDiet tweaks

Compare with previous version

marked this merge request as ready

changed title from Draft: Lower limits on breast shrinkage to prDiet minor tweaks

changed the description

merged

mentioned in commit e8f827a7