Remove alpha filler #3523

Merged
merged 1 commit into from Aug 13, 2015

Conversation

Projects
None yet
2 participants
@JakubValtar
Contributor

JakubValtar commented Jul 31, 2015

No longer needed because primary surface is now ARGB

f3b72c7

Remove alpha filler
No longer needed because primary surface is now ARGB

f3b72c7
@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Aug 5, 2015

Member

Are you absolutely certain we can do this? I've had to fix and re-fix this so many times that I've lost track.

Member

benfry commented Aug 5, 2015

Are you absolutely certain we can do this? I've had to fix and re-fix this so many times that I've lost track.

@JakubValtar

This comment has been minimized.

Show comment
Hide comment
@JakubValtar

JakubValtar Aug 5, 2015

Contributor

It's not needed as long as the surface has 4 bands (colors + alpha). Maybe we can leave it here as a safety measure? I wonder how this affects the performance (probably not at all).

Contributor

JakubValtar commented Aug 5, 2015

It's not needed as long as the surface has 4 bands (colors + alpha). Maybe we can leave it here as a safety measure? I wonder how this affects the performance (probably not at all).

benfry added a commit that referenced this pull request Aug 13, 2015

@benfry benfry merged commit 12956e0 into processing:master Aug 13, 2015

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Aug 13, 2015

Member

Fingers crossed...

Member

benfry commented Aug 13, 2015

Fingers crossed...

@JakubValtar JakubValtar deleted the JakubValtar:remove-alpha-filler branch Aug 15, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment