Skip to content
This repository has been archived by the owner on Dec 22, 2022. It is now read-only.

Use std::size_t in graph loader #132

Merged
merged 8 commits into from
Apr 18, 2022
Merged

Use std::size_t in graph loader #132

merged 8 commits into from
Apr 18, 2022

Conversation

maawad
Copy link
Member

@maawad maawad commented Apr 17, 2022

  • Fix a bug when NDEBUG is defined, causing calls inside asserts to get discarded
  • Assume std::size_t when loading market file
  • Throw exceptions for overflow/underflow cases

@maawad maawad requested a review from neoblizz April 17, 2022 19:07
@neoblizz neoblizz merged commit c9626f0 into gunrock:dev Apr 18, 2022
@neoblizz neoblizz mentioned this pull request Apr 18, 2022
Closed
@maawad maawad deleted the bug-market-loader branch April 18, 2022 06:57
@neoblizz neoblizz added the 🐛 bug Something isn't working label Jun 26, 2022
@maawad maawad mentioned this pull request Jul 23, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@neoblizz neoblizz neoblizz approved these changes

Assignees
No one assigned
Labels
🐛 bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@maawad @neoblizz