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. 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.
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.

Back to Bug 1530213 Comment 30