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.

Sign up for GitHub

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

Jump to bottom

Fix implementation of *_REQUIRES_T() #40

Closed
19 tasks done
martinmoene opened this issue Mar 5, 2020 · 0 comments
Closed
19 tasks done

Fix implementation of *_REQUIRES_T() #40

martinmoene opened this issue Mar 5, 2020 · 0 comments
Projects
Improve coherence between lite projects

Comments

@martinmoene
Copy link
Owner

martinmoene commented Mar 5, 2020 •
edited
Loading

See enable_if on CppReference.

Plan: Change to:

#define xxx_REQUIRES_T(...) \
    , typename std::enable_if< (__VA_ARGS__), int >::type = 0

Progress:

  • lest (NA)
  • any-lite
  • atomic-lite (no code yet)
  • byte-lite (NA)
  • expected-lite
  • gsl-lite: see PR 238
  • invoke-lite (NA)
  • observer-ptr-lite
  • optional-fun-lite (NA)
  • optional-bare (NA)
  • optional-lite
  • ring-span-lite
  • scope-guard-lite (no code yet)
  • span-lite
  • status-value-lite (NA)
  • string-view-lite (NA)
  • type-lite
  • value-ptr-lite
  • variant-lite
martinmoene added a commit to martinmoene/optional-lite that referenced this issue Mar 5, 2020
@martinmoene
Change requires to use a default template argument (#55, thanks to @i…
5b2fb1e 
…mprobablejan)

Fixes implementation of optional_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/optional-lite that referenced this issue Mar 5, 2020
@martinmoene
Change requires to use a default template argument (#55, thanks to @i…
68e7011 
…mprobablejan)

Fixes implementation of optional_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/optional-lite that referenced this issue Mar 5, 2020
@martinmoene
Change requires to use a default template argument (#55, thanks to @i…
e5b0ad5 
…mprobablejan)

Fixes implementation of optional_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/optional-lite that referenced this issue Mar 5, 2020
@martinmoene
Change requires to use a default template argument (#55, thanks to @i…
ca3402c 
…mprobablejan)

Fixes implementation of optional_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/optional-lite that referenced this issue Mar 5, 2020
@martinmoene
Change requires to use a default template argument (#55, thanks to @i…
af750b5 
…mprobablejan)

Fixes implementation of optional_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
@martinmoene martinmoene mentioned this issue Mar 5, 2020
Merged
martinmoene added a commit to martinmoene/optional-lite that referenced this issue Mar 5, 2020
@martinmoene
Change requires to use a default template argument (#55, thanks to @i…
32fd95a 
…mprobablejan) (#57)

Fixes implementation of optional_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/any-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
b03dcd7 
…ct 40)

Fixes implementation of any_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/expected-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
0d28717 
…ct 40)

Fixes implementation of nsel_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/observer-ptr-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
6d4a178 
…ct 40)

Fixes implementation of nsop_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to gsl-lite/gsl-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
0733f31 
…ct 40)

Fixes implementation of gsl_REQUIRES_T_(), see martinmoene/nonstd-lite-project#40 and https://en.cppreference.com/w/cpp/types/enable_if#Notes
@martinmoene martinmoene mentioned this issue Mar 6, 2020
Closed
martinmoene added a commit to martinmoene/ring-span-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
b59744e 
…ct 40)

Fixes implementation of nsrs_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/span-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
f4d273d 
…ct 40)

Fixes implementation of span_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/type-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
9b9af15 
…ct 40)

Fixes implementation of type_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/value-ptr-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
f493bf9 
…ct 40)

Fixes implementation of nsvp_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
martinmoene added a commit to martinmoene/variant-lite that referenced this issue Mar 6, 2020
@martinmoene
Change requires to use a default template argument (nonstd-lite-proje…
a0a1be0 
…ct 40)

Fixes implementation of variant_REQUIRES_T(), see martinmoene/nonstd-lite-project#40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Improve coherence between lite projects
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@martinmoene