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.

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

Fix for: Warnings with gcc 8 and earlier #2972 #2973

Merged
merged 1 commit into from
May 7, 2021
Merged

Fix for: Warnings with gcc 8 and earlier #2972 #2973

merged 1 commit into from
May 7, 2021

Conversation

tstuefe
Copy link
Contributor

@tstuefe tstuefe commented May 6, 2021

Hi,

Please consider this fix for a build warning with gcc 8 and earlier. This causes problems in OpenJDK, see issue for details.

Thank you!

..Thomas

@behdad
Copy link
Member

behdad commented May 6, 2021

Thanks Thomas. I'll review tomorrow.

@behdad behdad merged commit b54d9b6 into harfbuzz:master May 7, 2021
@tstuefe
Copy link
Contributor Author

tstuefe commented May 8, 2021

Thanks @behdad !

sezero added a commit to libsdl-org/SDL_ttf that referenced this pull request Jan 26, 2022
harfbuzz/harfbuzz#2973
harfbuzz/harfbuzz#2972
harfbuzz/harfbuzz@b54d9b6

Fixes the following warning:

In file included from ../external/harfbuzz-2.8.0/src/hb-ot-shape-complex-use.cc:33:0:
hb-ot-shape-complex-use-machine.rl: In instantiation of 'machine_index_t<Iter>::machine_index_t(const machine_index_t<Iter>&) [with Iter = hb_zip_iter_t<hb_iota_iter_t<unsigned int, unsigned int>, hb_filter_iter_t<hb_filter_iter_t<hb_zip_iter_t<hb_iota_iter_t<unsigned int, unsigned int>, hb_array_t<hb_glyph_info_t> >, find_syllables_use(hb_buffer_t*)::<lambda(const hb_glyph_info_t&)>, const<anonymous struct>&, 0u>, find_syllables_use(hb_buffer_t*)::<lambda(hb_pair_t<unsigned int, const hb_glyph_info_t&>)>, const<anonymous struct>&, 0u> >]':
hb-ot-shape-complex-use-machine.rl:249:4:   required from here
hb-ot-shape-complex-use-machine.rl:194:2: warning: base class 'struct hb_iter_with_fallback_t<machine_index_t<hb_zip_iter_t<hb_iota_iter_t<unsigned int, unsigned int>, hb_filter_iter_t<hb_filter_iter_t<hb_zip_iter_t<hb_iota_iter_t<unsigned int, unsigned int>, hb_array_t<hb_glyph_info_t> >, find_syllables_use(hb_buffer_t*)::<lambda(const hb_glyph_info_t&)>, const<anonymous struct>&, 0u>, find_syllables_use(hb_buffer_t*)::<lambda(hb_pair_t<unsigned int, const hb_glyph_info_t&>)>, const<anonymous struct>&, 0u> > >, hb_pair_t<unsigned int, hb_pair_t<unsigned int, hb_glyph_info_t&> > >' should be explicitly initialized in the copy constructor [-Wextra]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants