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

Using namespace dev #2505

Merged
merged 36 commits into from
Apr 10, 2023
Merged

Using namespace dev #2505

merged 36 commits into from
Apr 10, 2023

Conversation

cvvergara
Copy link
Member

Changes proposed in this pull request:

  • Wrap C++ code with namespace dev { code }
  • Adjusted code to use namespace directives (i.e. using pgrouting::Path)
  • Fix clang warnings
  • Cleanup of locale comments

@pgRouting/admins

cvvergara added 30 commits April 8, 2023 20:17
@cvvergara
[C++] adding missing namespace pgrouting
5ccf396
@cvvergara
[doc] included strings to files & space adjustment on version result
97fbec7
@cvvergara
[allpairs] Adding: use namespace directive
dd6529b
@cvvergara
[alpha_shape] Adding: use namespace directive
cf17d1a
@cvvergara
[astar] Adding: use namespace directive
bf6f1ef
@cvvergara
[bdAstar] Adding: use namespace directive
ee033ef
@cvvergara
[bdDijkstra] Adding: use namespace directive
ed0614f
@cvvergara
[bellman_ford] Adding: use namespace directive
b87e8ba
@cvvergara
[breadthFirstSearch] Adding: use namespace directive
d94fccd
@cvvergara
[chinese] Adding: use namespace directive
5e795c1
@cvvergara
[circuits] Adding: use namespace directive
5efde40
@cvvergara
[coloring] Adding: use namespace directive
22feef5
@cvvergara
[components] Adding: use namespace directive
b521c22
@cvvergara
[contraction] Adding: use namespace directive
ff96217
@cvvergara
[cpp_common] Adding: use namespace directive
d4a0054
@cvvergara
[dagShortestPath] Adding: use namespace directive
4bf6515
@cvvergara
[dijkstra] Adding: use namespace directive
6d4c2fb
@cvvergara
[dominator] Adding: use namespace directive
396d2ce
@cvvergara
[driving_distance] Adding: use namespace directive
108618d
@cvvergara
[ksp] Adding: use namespace directive
3377658
@cvvergara
[lineGraph] Adding: use namespace directive
25c2ee9
@cvvergara
[max_flow] Adding: use namespace directive
cd30481
@cvvergara
[mincut] Adding: use namespace directive
219b84d
@cvvergara
[ordering] Adding: use namespace directive
f25e5db
@cvvergara
[pickDeliver] Adding: use namespace directive
c2fd60f
@cvvergara
[planar] Adding: use namespace directive
315ff4b
@cvvergara
[spanningTree] Adding: use namespace directive
883295b
@cvvergara
[topologicalSort] Adding: use namespace directive
96a53e1
@cvvergara
[transitiveClosure] Adding: use namespace directive
8607d57
@cvvergara
[traversal] Adding: use namespace directive
ec57e75
@cvvergara cvvergara added this to the Release 3.6.0 milestone Apr 10, 2023
@cvvergara cvvergara merged commit 8faf4f7 into pgRouting:develop Apr 10, 2023
@cvvergara cvvergara deleted the using-namespace-dev branch May 10, 2023 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robe2 robe2 robe2 approved these changes

@dkastl dkastl Awaiting requested review from dkastl

@cayetanobv cayetanobv Awaiting requested review from cayetanobv

@omshinde omshinde Awaiting requested review from omshinde

@krashish8 krashish8 Awaiting requested review from krashish8

Assignees
No one assigned
Labels
C/C++ clang
Projects
None yet
Milestone
Release 3.6.0
Development

Successfully merging this pull request may close these issues.
2 participants
@cvvergara @robe2