Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FlowBlur: include <algorithm> #1

Closed
wants to merge 1 commit into from
Closed

FlowBlur: include <algorithm> #1

wants to merge 1 commit into from

Conversation

HolyWu
Copy link
Contributor

@HolyWu HolyWu commented Oct 5, 2014

Wanted by MSVC/ICC.

@dubhater
Copy link
Owner

dubhater commented Oct 5, 2014

Can you change that commit message to something like

FlowBlur: include <algorithm>

Wanted by MSVC/ICC.

please?

@HolyWu HolyWu changed the title Include <algorithm> in MVFlowBlur.cpp FlowBlur: include <algorithm> Oct 5, 2014
@HolyWu
Copy link
Contributor Author

HolyWu commented Oct 5, 2014

OK.

@dubhater
Copy link
Owner

dubhater commented Oct 6, 2014

That was the title of the pull request, not the commit message.
The thing that needs to change is Compilation will fail in MSVC or ICC on Windows without algorithm inc… …luded

@HolyWu
Copy link
Contributor Author

HolyWu commented Oct 6, 2014

Sorry, I can't find the place to let me edit the comment of existing commit.

@dubhater
Copy link
Owner

dubhater commented Oct 6, 2014

It can be done with git, but nevermind. Pushed your patch with edited commit message.

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

Successfully merging this pull request may close these issues.

2 participants