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.

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

Fix #940: saveBlend() handles zero or one blend only #941

Merged
merged 4 commits into from
Aug 30, 2019
Merged

Conversation

blueshade7
Copy link
Contributor

@blueshade7 blueshade7 commented Aug 29, 2019

fixes #940

@@ -974,15 +974,16 @@ static void saveBlend(cfrCtx h, float *realValue, abfOpEntry *blendEntry) {
// copy the defaul region value
blendValues[0] = defaultValue = INDEX_REAL(0);

/* now, copy the blend abusolate values to blendArray[i].
/* now, copy the blend absolate values to blendArray[i].
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean absolute?

(Otherwise these changes looks fine to me.)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I didn't realize there are two typos in one word... will fix

Copy link
Contributor

@cjchapman cjchapman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

@miguelsousa miguelsousa merged commit 97eb224 into develop Aug 30, 2019
@miguelsousa miguelsousa deleted the fix_940 branch August 30, 2019 00:42
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.

mismatch between cffread.c:saveBlend() and cffwrite_dict.c:saveRealBlendOp()
3 participants