Yeah, as I said, it wouldn't look too nice in this case. However something like
```
if(gainValue > 0)
   gainValue = "+" + gainValue;

gainValue += "%";
```
would be cleaner imho, but one could argue it's a value change more. but as mentioned before, I don't think that's really something we have to change, since it's pretty much a personal preference and in the end would be defined by code guidelines we don't have yet.
Thanks for the patch!