Sorry yes mailing list is what I meant. I'll update the comments with proper punctuation and the word changes.
> ...
This patch:
currentIndex; i < this.mMaxAttendees - 1
It seems that having the - 1 there has no affect on the code even with the < instead of <= so I will remove it. To be honest I'm having a hard time remembering why I added it in the first place.
I'll do some proofreading and up load an edited patch.
Bug 1530213 Comment 30 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Sorry yes mailing list is what I meant. I'll update the comments with proper punctuation and the word changes.
> ...
This patch:
currentIndex; i < this.mMaxAttendees - 1
It seems that having the - 1 there has no affect on the code even with the < instead of <= so I will remove it.
My initial thought was that by subtracting one from mMaxAttendees it would prevent the loop from running one too many times. Of course that's what removing the "=" would do as well.
I'll do some proofreading and up load an edited patch.