8000 Replaced url filtering with linq query, Replaced Click delegate with lambda expression by BasedMan · Pull Request #68 · alexbakker/Toxy · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Sep 30, 2018. It is now read-only.

Replaced url filtering with linq query, Replaced Click delegate with lambda expression #68

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Replaced url filtering with linq query, Replaced Click delegate with lambda expression #68

wants to merge 1 commit into from

Conversation

BasedMan
Copy link
Contributor

Minor cosmetic changes

@BasedMan
Replaced url filtering with linq query
0721226 
Replaced Click delegate with lambda expression
@alexbakker
Copy link
Owner

Does LINQ somehow improve performance here?
I personally think the foreach loop is more readable in this case. Does anyone else have an opinion on this?

@BasedMan
Copy link
Contributor Author

I think LINQ is slightly slower than foreach, although the difference in this case is negligible. The reason I did it was because to me it is more readable and it gets rid of the "parts" array (although obviously this could also be done with foreach). How it should be done is mostly down to personal preference :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BasedMan @alexbakker
0