Skip to content

Sync a couple more changes#127

Merged
jmacnak merged 3 commits into
google:mainfrom
jmacnak:sync
May 19, 2026
Merged

Sync a couple more changes#127
jmacnak merged 3 commits into
google:mainfrom
jmacnak:sync

Conversation

@jmacnak
Copy link
Copy Markdown
Member

@jmacnak jmacnak commented May 18, 2026

No description provided.

kocdemir and others added 2 commits May 18, 2026 15:44
Ensure device pose window won't call fatal errors and avoid GL rendering
instead when trying to get color buffer contents for rendering foldable
devices.

Bug: 477193075
Test: manual, use device pose extended window controls with -feature GuestAngle
Change-Id: Ic1436647ad4fe4bdaaa2ff0c61a985d7513eb723
Fixing these warnings either by adding a reference to the loop index's
type or removing the reference from it.

hardware/google/gfxstream/tests/end2end/TestDataUtils.cpp:65:29: error:
loop variable 'possiblePath' of type 'const std::string &'
(aka 'const basic_string<char> &') binds to a temporary constructed from
type 'reference' (aka 'const std::filesystem::path &')
[-Werror,-Wrange-loop-construct]
   65 |     for (const std::string& possiblePath : possiblePaths) {
      |                             ^
hardware/google/gfxstream/tests/end2end/TestDataUtils.cpp:65:10: note: use
non-reference type 'std::string' (aka 'basic_string<char>') to make
construction explicit or type 'const std::filesystem::path &' to prevent
copying
   65 |     for (const std::string& possiblePath : possiblePaths) {
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Bug: 153747076
Test: m & presubmits
Flag: EXEMPT refactor
Change-Id: I457af4828122e9942f5f8508f7cbaf24bc596e22
@jmacnak jmacnak requested a review from kocdemir May 18, 2026 22:51
Deleted in ag/39370045 as part of merge conflict resolution?
@jmacnak jmacnak added this pull request to the merge queue May 19, 2026
Merged via the queue into google:main with commit c96ea7f May 19, 2026
9 checks passed
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.

3 participants