Skip to content

video/gtk4: Move the dmabuf cfg to the correct bracket level

Jordan Petridіs requested to merge alatiera/gst-plugins-rs:alatiera/gst-gl-2 into main

This was defined one bracket above, which was causing the gst-gl codepath below to also be disabled when there was no dmabuf feature enabled.

This was also resulting in the following warning as we were never creating the MappedFrame::GL vartiant due to this

warning: unused variable: `wrapped_context`
   --> video/gtk4/src/sink/frame.rs:541:85
    |
541 | ...", feature = "gst-gl"))] wrapped_context: Option<
    |                             ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_wrapped_context`
    |
    = note: `#[warn(unused_variables)]` on by default

warning: variant `GL` is never constructed
  --> video/gtk4/src/sink/frame.rs:80:5
   |
74 | enum MappedFrame {
   |      ----------- variant in this enum
...

Move the cfg to the appropriate place where it encaplsulates only the dmabuf related code.

Merge request reports