Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Add image orientation option #45

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

Add image orientation option #45

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 9, 2016

Come phones returning images in rotated faction so now there is possibility to have picture (byte array) and angle of that picture so it is very simple to rotate that image without additional work.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@ghost
Copy link
Author

ghost commented Nov 9, 2016

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes and removed cla: no labels Nov 9, 2016
@@ -457,6 +486,7 @@ public void writeToParcel(Parcel out, int flags) {
out.writeParcelable(ratio, 0);
out.writeByte((byte) (autoFocus ? 1 : 0));
out.writeInt(flash);
out.writeByte((byte) (autoFocus ? 1 : 0));
Copy link

Choose a reason for hiding this comment

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

Shouldn't it be
out.writeByte((byte) (computeAngle ? 1 : 0));
?

ewolfox pushed a commit to ewolfox/cameraview that referenced this pull request Apr 13, 2018
Ignore tap-to-focus if not supported
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants