Changeset 95645 in vbox for trunk/src/VBox/Main/include/ConsoleImpl.h

Timestamp:

Jul 14, 2022 10:33:14 AM (3 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

152289

Message:

Recording/Main: Simplified and got rid of some destruction races by not dynamically (re-)allocating the recording context. Added docs, a bit of renaming. bugref:9286

File:

• trunk/src/VBox/Main/include/ConsoleImpl.h (modified) (2 diffs)

trunk/src/VBox/Main/include/ConsoleImpl.h

@@ -174 +174 @@
     AudioVideoRec *i_recordingGetAudioDrv(void) const { return mRecording.mAudioRec; }
 # endif
-    RecordingContext *i_recordingGetContext(void) const { return mRecording.mpCtx; }
+    RecordingContext *i_recordingGetContext(void) { return &mRecording.mCtx; }
 # ifdef VBOX_WITH_AUDIO_RECORDING
     HRESULT i_recordingSendAudio(const void *pvData, size_t cbData, uint64_t uDurationMs);
@@ -1156 +1156 @@
     {
         Recording()
-            : mpCtx(NULL)
 # ifdef VBOX_WITH_AUDIO_RECORDING
-            , mAudioRec(NULL)
+            : mAudioRec(NULL)
 # endif
         { }
 
         /** The recording context. */
-        RecordingContext     *mpCtx;
+        RecordingContext      mCtx;
 # ifdef VBOX_WITH_AUDIO_RECORDING
         /** Pointer to capturing audio backend. */