fdkaacenc: Update documentation to clarify bitrate and peak-bitrate
authorSanchayan Maity <sanchayan@asymptotic.io>
Sat, 29 Oct 2022 10:47:53 +0000 (16:17 +0530)
committerSanchayan Maity <sanchayan@asymptotic.io>
Sun, 30 Oct 2022 11:24:51 +0000 (16:54 +0530)
bitrate property is only applicable for constant bitrate and
peak-bitrate is only applicable for variable bitrate. Clarify
the same.

subprojects/gst-plugins-bad/docs/plugins/gst_plugins_cache.json
subprojects/gst-plugins-bad/ext/fdkaac/gstfdkaacenc.c

index bbfd088..2e24fc2 100644 (file)
                         "writable": true
                     },
                     "bitrate": {
-                        "blurb": "Target Audio Bitrate (0 = fixed value based on  sample rate and channel count)",
+                        "blurb": "Target Audio Bitrate. Only applicable if rate-control=cbr. (0 = fixed value based on sample rate and channel count)",
                         "conditionally-available": false,
                         "construct": false,
                         "construct-only": false,
                         "writable": true
                     },
                     "peak-bitrate": {
-                        "blurb": "Peak Bitrate to adjust maximum bits per audio frame. Bitrate is in bits/second. (0 = Not set)",
+                        "blurb": "Peak Bitrate to adjust maximum bits per audio frame. Bitrate is in bits/second. Only applicable if rate-control=vbr. (0 = Not set)",
                         "conditionally-available": false,
                         "construct": false,
                         "construct-only": false,
                         "readable": true,
                         "type": "gint",
                         "writable": true
-                     },
+                    },
                     "rate-control": {
-                        "blurb": "Whether Constant or Variable Bitrate should be used",
+                        "blurb": "Whether Constant or Variable Bitrate should be used.",
                         "conditionally-available": false,
                         "construct": false,
                         "construct-only": false,
                         "writable": true
                     },
                     "vbr-preset": {
-                        "blurb": "AAC Variable Bitrate configurations",
+                        "blurb": "AAC Variable Bitrate configurations. Requires rate-control as vbr.",
                         "conditionally-available": false,
                         "construct": false,
                         "construct-only": false,
         "tracers": {},
         "url": "Unknown package origin"
     }
-}
+}
\ No newline at end of file
index bb468c9..abbf5d1 100644 (file)
@@ -749,8 +749,8 @@ gst_fdkaacenc_class_init (GstFdkAacEncClass * klass)
   g_object_class_install_property (object_class, PROP_BITRATE,
       g_param_spec_int ("bitrate",
           "Bitrate",
-          "Target Audio Bitrate (0 = fixed value based on "
-          " sample rate and channel count)",
+          "Target Audio Bitrate. Only applicable if rate-control=cbr. "
+          "(0 = fixed value based on sample rate and channel count)",
           0, G_MAXINT, DEFAULT_BITRATE,
           G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
 
@@ -765,7 +765,7 @@ gst_fdkaacenc_class_init (GstFdkAacEncClass * klass)
       g_param_spec_int ("peak-bitrate",
           "Peak Bitrate",
           "Peak Bitrate to adjust maximum bits per audio frame. "
-          "Bitrate is in bits/second. (0 = Not set)",
+          "Bitrate is in bits/second. Only applicable if rate-control=vbr. (0 = Not set)",
           0, G_MAXINT, DEFAULT_PEAK_BITRATE,
           G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));