stotko / stdgpu

stdgpu: Efficient STL-like Data Structures on the GPU
https://stotko.github.io/stdgpu/
Apache License 2.0
1.15k stars 81 forks source link

src: Remove forward declaration headers #354

Closed stotko closed 1 year ago

stotko commented 1 year ago

In addition to the usual headers, there are additional headers for all containers containing forward declarations of them. Although this may in some cases be useful to reduce coupling and compile times, they also require maintenance and hide the default parameters from the actual headers. Furthermore, most users will likely include the full headers anyways and in the future C++20 modules will solve the include problem in a much cleaner way. Thus, remove the forward declaration headers.

This also partially addresses our current inconsistent header name convention discussed in #38.

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (bb103bd) 97.14% compared to head (b76c99c) 97.14%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #354 +/- ## ======================================= Coverage 97.14% 97.14% ======================================= Files 31 31 Lines 2344 2344 ======================================= Hits 2277 2277 Misses 67 67 ``` | [Impacted Files](https://codecov.io/gh/stotko/stdgpu/pull/354?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko) | Coverage Δ | | |---|---|---| | [src/stdgpu/bitset.cuh](https://codecov.io/gh/stotko/stdgpu/pull/354?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9iaXRzZXQuY3Vo) | `100.00% <ø> (ø)` | | | [src/stdgpu/deque.cuh](https://codecov.io/gh/stotko/stdgpu/pull/354?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9kZXF1ZS5jdWg=) | `100.00% <ø> (ø)` | | | [src/stdgpu/mutex.cuh](https://codecov.io/gh/stotko/stdgpu/pull/354?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS9tdXRleC5jdWg=) | `100.00% <ø> (ø)` | | | [src/stdgpu/unordered\_map.cuh](https://codecov.io/gh/stotko/stdgpu/pull/354?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS91bm9yZGVyZWRfbWFwLmN1aA==) | `100.00% <ø> (ø)` | | | [src/stdgpu/unordered\_set.cuh](https://codecov.io/gh/stotko/stdgpu/pull/354?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS91bm9yZGVyZWRfc2V0LmN1aA==) | `100.00% <ø> (ø)` | | | [src/stdgpu/vector.cuh](https://codecov.io/gh/stotko/stdgpu/pull/354?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko#diff-c3JjL3N0ZGdwdS92ZWN0b3IuY3Vo) | `100.00% <ø> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Patrick+Stotko)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.