Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Use std::find in atBuiltIn #3

Merged
merged 1 commit into from
May 4, 2014

Conversation

kanielc
Copy link
Collaborator

@kanielc kanielc commented May 4, 2014

  • Made builtIns vector static
  • Use std::find instead of hand-rolled loop

Sorry about the EOL whitespace removal. My vim settings were really
aggressive.

I can undo them if you wish.

Sorry about the EOL whitespace removal.  My vim settings were really
aggressive.

I can undo them if you wish.
@kanielc kanielc changed the title Use std::algorithm in atBuiltIn Use std::find in atBuiltIn May 4, 2014
JossWhittle added a commit that referenced this pull request May 4, 2014
@JossWhittle JossWhittle merged commit 12cca04 into JossWhittle:master May 4, 2014
@JossWhittle
Copy link
Owner

Nice catch, I think there are a few other places where this could used, i'll have a look through tomorrow and see if I can catch them. Cheers :)

@kanielc
Copy link
Collaborator Author

kanielc commented May 4, 2014

I saw a few more in that file, I'll get on them.

On Sat, May 3, 2014 at 9:19 PM, Joss Whittle [email protected]:

Nice catch, I think there are a few other places where this could used,
i'll have a look through tomorrow and see if I can catch them. Cheers :)


Reply to this email directly or view it on GitHubhttps://github.com//pull/3#issuecomment-42121073
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants