stdlib
stdlib copied to clipboard
fortran-lang
Stdlib linked list
This PR is to submit the first draft of the linked list module for stdlib. This is based upon #68 by Milan, #463 by me.
I have been working on this module as a part of my GSoC Project.
All my work was regularly updated on my personal repository that I have been sharing with everyone in my weekly report.
Link to my repository (here).
A bit detailed project report is available (here).
List of all the weekly Blogs - (here)
The module was tested in two compilers, intel oneAPI and gfortran.
How should we review this patch best? While there is an implementation I don't find a specification, examples or tests in this PR.
Hi Sebastian,
Chetan and I can work on that - the documentation resides in his own repository together with the workplan, but let us distill a version that is suitable for reviewing the patch. We have a meeting planned for this friday, mainly for preparing the conference, but this may be a second topic.
Op ma 13 sep. 2021 om 23:29 schreef Sebastian Ehlert < @.***>:
How should we review this patch best? While there is an implementation I don't find a specification, examples or tests in this PR.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/fortran-lang/stdlib/pull/491#issuecomment-918592594, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAN6YR3ZNUGNWGZWTMDXGTTUBZUKTANCNFSM5CRFTFJA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
I have not been in a position yet to follow on this. I want to contact @chetan karwa @.***> about it, but my workload has not diminished enough that I can actually spare the energy yet.
Op zo 19 dec. 2021 om 13:46 schreef Sebastian Ehlert < @.***>:
Any update on this feature?
— Reply to this email directly, view it on GitHub https://github.com/fortran-lang/stdlib/pull/491#issuecomment-997386216, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAN6YR4ZYWPHQOBJUUCMVNDURXHZBANCNFSM5CRFTFJA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID: @.***>
I guess this PR has become stale. Since we already have a decent implementation here, should we try to update the PR, address the review comments and get it merged?
I guess this PR has become stale. Since we already have a decent implementation here, should we try to update the PR, address the review comments and get it merged?
Maybe @arjenmarkus / @chetan has some news/updates. Specs and tests are mainly missing.
I have (finally, sigh ;)) started setting up a test program. The documentation will follow with it.