Feat/camera x

[KryptKode]

Notes

• to handle the media storage location and image/video capture intents according to the developer guides for image/video

  • add MediaOutputHelper
    • to create OutputStream for photos
    • to create FileDescriptor for videos (currently requires API 26+). The API might change
    • add support for specifying the output URI if present in the intent
    • when the output URI is specified,
      • for Image Capture, we return a Bitmap as a data extra and also the URI as the Intent data
      • for Video Capture we return the Uri as the Intent data
    • if no output URI is specified in the capture intent or if there is an error while trying to access the URI, use the default location with MediaStore, so we do not return inconsistent URIs (eg, SAF tree URIs)

• to test the image/video capture intents, you can download this APK created from this repository (tried to use ADB but it didn't work well)

• handle torch state in video capture

• handle some camera errors (documented here)

  • errors during camera lifecycle
  • when capturing images
  • when recording videos

• add CameraXInitializer to abstract CameraXPreview initialisation logic

[tibbi]

again, Im not testing functionality yet, just checking code style. You are often squeezing it too much though, doing many things at once on 1 line. It is unreadable though and if it crashes somewhere, I wont know which part crashed it. Short code doesnt mean good code, just make it more readable by creating local variables, moving brackets on new lines etc.

[tibbi]

are you fixing the issues, or just at the final refactoring?

[KryptKode]

are you fixing the issues, or just at the final refactoring?

Will push the fixes for this, then raise other PRs for the remaining stuffs

[tibbi]

ok, guess this is good enough to be accepted