Skip to content

added for_each unit test#54

Open
vucinicv wants to merge 2 commits intonsubtil:masterfrom
vucinicv:unit_tests/for_each
Open

added for_each unit test#54
vucinicv wants to merge 2 commits intonsubtil:masterfrom
vucinicv:unit_tests/for_each

Conversation

@vucinicv
Copy link
Contributor

No description provided.

parallel<system>::fill(data, 0);
parallel<system>::for_each(data.size(), add<system, int>(data, sub_num));

for (int i = 0; i < data.size(); i++)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

@nsubtil
Copy link
Owner

nsubtil commented Feb 18, 2016

Mostly looks good, but the warnings I pointed out need to be fixed prior to merging this.

@nsubtil
Copy link
Owner

nsubtil commented Feb 22, 2016

This needs to be rebased and the history cleaned up prior to merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants