optimize for loop in operations.py

[virophagesp]

the order of the list it is iterating over does not matter

Why is this change needed: it reduces latency for reply scanning

Additional Info: the for loop doesnt require a proper order

Checklist:

• [x] My code follows the style guidelines of this project and I have read CONTRIBUTING.md
• [x] I have performed a self-review of my own code
• [x] I have commented my code, particularly in hard-to-understand areas
• [x] I have made corresponding changes to the documentation
• [x] My changes generate no new warnings
• [x] Any dependent changes have been merged and published in downstream modules
• [x] I have checked my code and corrected any misspellings

[virophagesp]

how do i check the boxes

[virophagesp]

nevermind i figured it out

[virophagesp]

@ThioJoe this is ready to be checked

[ThioJoe]

Isn't this made redundant by line 744?

[virophagesp]

not exactly, line 744 prevents it going through if it never had the @ symbol in the first place line 749 modifies the original commentText variable allThreadAuthorNames can be very long usually there is only a single @ username when commentText gets modified to no longer have any @'s my code breaks out of the loop

[virophagesp]

I am going to benchmark the code and find how much of an improvement it is

[virophagesp]

so turns out it is less than 0.05% in most cases I did the original benchmark wrong I'm gonna close this now