FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix out-by-one error by diarmidmackenzie · Pull Request #5790 · aframevr/aframe · GitHub

Fix out-by-one error - #5790

Merged
dmarcos merged 1 commit into
aframevr:masterfrom
diarmidmackenzie:issue-5789
Feb 21, 2026
Merged

Fix out-by-one error#5790
dmarcos merged 1 commit into
aframevr:masterfrom
diarmidmackenzie:issue-5789

Conversation

diarmidmackenzie commented Feb 5, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Description:

As described in issue #5789

Changes proposed:

Fix out by 1 error, so pixel row 0 swaps with row (height -1) not (height) and vice-versa.
Also simplified code & reduced calculations by pre-computing the start and end indices.

Testing

  • "screenshot" UTs run clean. They don't test in enough detail to notice any change.
  • Live.

Before fix: I took a screenshot from the Anime UI example and opened in Paint. You can see a single layer of blank pixels at the top of the image:

After fix: no blank row of pixels at the top or bottom of the image:

diarmidmackenzie marked this pull request as draft February 5, 2026 15:24
diarmidmackenzie marked this pull request as ready for review February 5, 2026 15:24

arpu commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

works

dmarcos commented Feb 21, 2026

Copy link
Copy Markdown
Member

Thanks!

dmarcos merged commit d2e9475 into aframevr:master Feb 21, 2026
1 check passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL