New pull request for lambda functions#114
Conversation
|
Co-authored-by: Victor Eijkhout eijkhout@tacc.utexas.edu |
|
@vulder I added the assert example again. |
|
refs #35 |
vulder
left a comment
There was a problem hiding this comment.
LGTM so far, I put it onto the tentatively ready topics for the next SG20 meeting. However, we should resolve the open issue until then.
| #### Points to cover | ||
|
|
||
| _This section lists important details for each point._ | ||
|
|
There was a problem hiding this comment.
There are currently no points to cover in this draft. We should probably add at least the core ones to cover the necessary skills that are needed to bring a student to the point where they can fulfill all ASSBATs.
| #### Background/Required Knowledge | ||
|
|
||
| A student: | ||
| _TODO: Add cross-reference to these |
There was a problem hiding this comment.
There is still an open TODO here.
|
|
||
| A student should be able to: | ||
|
|
||
| 1. Utilize `std::function` as a means to hold and transfer lambdas |
There was a problem hiding this comment.
We should probably not advertise std::function here but the better alternatives of move_only_function and copyable_function.
| 5. Specify the return type for a lamda function, if not detected by the compiler | ||
| 5. Use a lambda in a class, capturing and utilizing class data via `this` | ||
|
|
||
| #### Caveats |
There was a problem hiding this comment.
Seeing std::function here, we should probably put a caveat here that highlights the problematic interaction between lambdas and std::function, where one is by default const and the other is not const correct.
| X x = [c = b](){ if (c) return X(5, 10) else X("a", "b"); }(); | ||
| ``` | ||
|
|
||
| Todo add caption |
| #### Caveats | ||
|
|
||
| #### Points to cover | ||
|
|
There was a problem hiding this comment.
The knowledge/content for a few of the ASSBATs is still missing in the points to cover.
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
Co-authored-by: Florian Sattler <vuld3r@gmail.com>
@vulder @jcvw The North America subgroup finished the lambda functions.
Please have a look and make comments.