Notice: If the game stops working after an update, it might be a savegame incompatibility. Remove your savegame folder and try again before posting about it.
Registration has an extra hurdle now: There have been advertisement bots, so I added a security keyword to registration. Deal with it.
  0 users browsing Bug Reports. | 1 bot  
Main » Bug Reports » UIList Issue not quite resolved it totes is
Pages: 1
Posted on 03-07-13, 11:04 pm (revision 1)

 

Posts: 71/105
Since: 12-06-12

Last post: 4026 days
Last view: 4007 days
Sorry to make another thread about this, but the other one was closed. Feel free to delete this one after it's been resolved.

Your fix for the UIList Index doesn't seem to work, as trying to select a new character's race and pressing the "go left" button while on 'Human' still causes an Array-Out-of-Bounds exception.


I tossed together a quick fix that seems to work. (Would have done this yesterday, but I was just being lazy :P )

Replace:
_index = value < Items.Count || value > -1 ? value : 0;

With:
_index = value < Items.Count ? value < 0 ? Items.Count-1 : value : 0;


I tested it out and it seems to behave nicely, so hopefully we can call this one truly and completely resolved.

Lead writer on Noxico (and sometimes programmer or designer) <br>
Posted on 03-08-13, 09:43 am
<i>Prophet of Celestia</i><br>Baka on the streets, senpai in the sheets

Posts: 177/344
Since: 06-08-12

Last post: 1696 days
Last view: 1235 days
Well, fuck. Implemented. Will push when the next work on lootsets is done.

I don't just program Noxico...
Posted on 03-14-13, 01:48 pm

 

Posts: 1/2
Since: 02-10-13

Last post: 4025 days
Last view: 3292 days
If I were you, I'd also add some parentheses to that double ternary operator. I don't think it'd change much how the compiler sees it, but it'd make it more readable.

Just a lil' suggestion :)
Posted on 03-14-13, 02:14 pm
<i>Prophet of Celestia</i><br>Baka on the streets, senpai in the sheets

Posts: 184/344
Since: 06-08-12

Last post: 1696 days
Last view: 1235 days
Done. Will push with the next update.

I don't just program Noxico...
Pages: 1
Main » Bug Reports » UIList Issue not quite resolved it totes is