MobileRoboticsSkoltech / OpenCamera-Sensors

Android app for synchronized recording of video and IMU data with advanced camera options, useful for 3D reconstruction, SLAM, AR, video stabilization. Supports remote control over network.
GNU General Public License v3.0
173 stars 19 forks source link

Fixes of RecSync replated to unused video files #54

Closed TimPushkin closed 2 years ago

TimPushkin commented 2 years ago

Because of the new two-staged video recording scheme video files may be created even if the recording is not going to be started. Such files remain unused and need to be deleted manually. That is what changes from this PR do.

TimPushkin commented 2 years ago

I personally don't really like the solution I came up with in the MainActivity where I added the settings_is_active field. It is set to true when settings fragment starts to open and set to false after it is closed. MainActivity already has settingsIsOpen() method but it is private and doesn't act as needed (it may return false when settings opening has been initiated).