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

Jump to bottom

Feature: Introduce writeSubData in Texture API #385

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Feature: Introduce writeSubData in Texture API #385

wants to merge 1 commit into from

Conversation

nemerle
Copy link
Contributor

@nemerle nemerle commented Aug 4, 2019

This is mostly a proposal and a call for help from D3D11 and Vulkan gurus out there.

What is missing:

  • Vulkan support
  • D3D11 support
  • Optional written-area-within-target checks.

@nemerle
Feature: Introduce writeSubData in Texture API
b655d7b 
This is mostly a proposal and a call for help from D3D11 and Vulkan gurus out there.

What is missing:
- Vulkan support
- D3D11 support
- Optional written-area-within-target checks.
@BearishSun
Copy link
Member

I'd prefer the new method is also named writeData and is just an overload. Other than that I think it's fine, as long as the underlying API's all support this.

@nemerle
Copy link
Contributor Author

nemerle commented Aug 5, 2019

I'd prefer the new method is also named writeData and is just an overload. Other than that I think it's fine, as long as the underlying API's all support this.

I've originally named those methods writeData, but was unsure how well the @copydoc worked with overridden functions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nemerle @BearishSun