Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

Bugfix for static_addressof issues. #76

Merged
merged 3 commits into from
Jan 8, 2015
Merged

Bugfix for static_addressof issues. #76

merged 3 commits into from
Jan 8, 2015

Conversation

Fraser999
Copy link
Contributor

No description provided.

Bugfix for missing "detail::" qualifier on static_addressof and added…
3b5459b 
… flag to allow use of std::addressof rather than detail::static_addressof.
@Fraser999 Fraser999 mentioned this pull request Jan 8, 2015
Closed
@viboes
Copy link
Collaborator

viboes commented Jan 8, 2015

I would prefer something like


template <typename T>
BOOST_CONSTEXPR T\* static_addressof(T& ref)
{
  return std::addressof(ref);
}

#else
...

@Fraser999
Copy link
Contributor Author

Should be updated now.

@Fraser999
Copy link
Contributor Author

Yes - makes sense. Updated again.

viboes added a commit that referenced this pull request Jan 8, 2015
@viboes
Merge pull request #76 from Fraser999/addressof_flag
c4d594d 
Bugfix for static_addressof issues.
@viboes viboes merged commit c4d594d into ptal:master Jan 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Fraser999 @viboes