SakulFlee/Orbital
1
0
Fork 0
Code Issues 51 Pull requests 24 Projects Releases 1 Packages Wiki Activity

fix: return error on texture data size mismatch instead of warning #716

Open
SakulFlee wants to merge 1 commit from fix/texture-size-mismatch-error into main
pull from: fix/texture-size-mismatch-error
merge into: SakulFlee:main
Conversation 0 Commits 1 Files changed 2 +34 -8
SakulFlee commented 2026-07-17 22:05:15 +02:00
Owner
Copy link

Previously, when texture pixel data size didn't match expected dimensions,
the code only logged a warning but proceeded to write the data anyway,
risking GPU buffer overflow or texture corruption. Now it returns a
DataSizeMismatch error with detailed information about the mismatch.

Previously, when texture pixel data size didn't match expected dimensions, the code only logged a warning but proceeded to write the data anyway, risking GPU buffer overflow or texture corruption. Now it returns a DataSizeMismatch error with detailed information about the mismatch.
fix: return error on texture data size mismatch instead of warning
Some checks failed
ci/woodpecker/push/linting Pipeline failed
Details
a3c2b8980e
 
Previously, when texture pixel data size didn't match expected dimensions,
the code only logged a warning but proceeded to write the data anyway,
risking GPU buffer overflow or texture corruption. Now it returns a
DataSizeMismatch error with detailed information about the mismatch.
Some checks are pending
ci/woodpecker/push/linting Pipeline failed
Details
ci/woodpecker/pr/test
Required
ci/woodpecker/pr/linting
Required
Some required checks are missing.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/texture-size-mismatch-error:fix/texture-size-mismatch-error
git switch fix/texture-size-mismatch-error
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Context: Async
Async specific, be it bug or feature

  
Context: Bug
Something isn't working as intended

  
Context: CI
GitHub Actions / CI related tasks

  
Context: Dependencies
Pull requests that update a dependency file

  
Context: Documentation
Improvements or additions to documentation

  
Context: Enhancement
New feature or request

  
Context: Example
Example related issues

  
Context: Macro

   
Context: Runtime
Runtime related issues

  
Difficulty: Easy

   
Difficulty: Hard

   
Difficulty: Medium

   
Platform: Android
ONLY if this is platform specific!

  
Platform: iOS
ONLY if this is platform specific!

  
Platform: Linux
ONLY if this is platform specific!

  
Platform: macOS
ONLY if this is platform specific!

  
Platform: Web
ONLY if this is platform specific!

  
Platform: Windows
ONLY if this is platform specific!

  
Type: Discussion

   
Type: Evaluation
Further evaluation is needed

  
Type: Tracker
Issues acting as trackers; I.e. a combination of issues of the same overall topic.

No labels
Context: Async
Context: Bug
Context: CI
Context: Dependencies
Context: Documentation
Context: Enhancement
Context: Example
Context: Macro
Context: Runtime
Difficulty: Easy
Difficulty: Hard
Difficulty: Medium
Platform: Android
Platform: iOS
Platform: Linux
Platform: macOS
Platform: Web
Platform: Windows
Type: Discussion
Type: Evaluation
Type: Tracker
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
SakulFlee
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
SakulFlee/Orbital!716
No description provided.