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

Phoneme cleanup #3356

Merged
merged 11 commits into from
Oct 13, 2021
Merged

Phoneme cleanup #3356

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
3ee14df
Changed ProcessSigns back to GetSigns. Use insert instead of find and…
gknisely Oct 13, 2021
39a97d6
Changed ProcessSigns back to GetSigns.
gknisely Oct 13, 2021
0a5af7b
updated route_num_type to is_route_num_type.
gknisely Oct 13, 2021
6e245e8
refactored name
gknisely Oct 13, 2021
070ea5e
refactored function name. ProcessSigns--->GetSigns
gknisely Oct 13, 2021
596ce70
renamed route_num_type to is_route_num_type
gknisely Oct 13, 2021
59bbeac
moved logic to a lambda function get_pronunciations
gknisely Oct 13, 2021
b97546e
code clean up. moved logic to get_pronunciations lambda function tha…
gknisely Oct 13, 2021
1d015d0
updated.
gknisely Oct 13, 2021
5e0b0f0
clean up
gknisely Oct 13, 2021
749d801
Update CHANGELOG.md
gknisely Oct 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
renamed route_num_type to is_route_num_type
  • Loading branch information
@gknisely
gknisely committed Oct 13, 2021
commit 596ce707c3379d52f2a116fc1a3a8fb6966b3082
2 changes: 1 addition & 1 deletion src/mjolnir/graphtilebuilder.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ GraphTileBuilder::GraphTileBuilder(const std::string& tile_dir,
// Create sign builders
for (uint32_t i = 0; i < header_->signcount(); i++) {
name_info.insert({signs_[i].text_offset()});
signs_builder_.emplace_back(signs_[i].index(), signs_[i].type(), signs_[i].route_num_type(),
signs_builder_.emplace_back(signs_[i].index(), signs_[i].type(), signs_[i].is_route_num_type(),
signs_[i].tagged(), signs_[i].text_offset());
}

Expand Down