Memory Leak

[Maklith]

Each screenshot operation causes an unmanaged memory leak

public static class ScreenCapture
{
    static DX11ScreenCaptureService screenCaptureService ;
    private static WeakReference _weakReference;
    static Queue<Bitmap> bitmaps = new();
    public static Queue<Bitmap> CaptureAllScreen()
    {
        screenCaptureService = new DX11ScreenCaptureService();
        foreach (var display in screenCaptureService.GetDisplays(screenCaptureService.GetGraphicsCards().First()))
        {
            var  screenCapture = screenCaptureService.GetScreenCapture(display);
            var fullscreen = screenCapture.RegisterCaptureZone(0, 0, screenCapture.Display.Width, screenCapture.Display.Height);
            while (!screenCapture.CaptureScreen()) { }
            SixLabors.ImageSharp.Configuration customConfig = SixLabors.ImageSharp.Configuration.Default.Clone();
            customConfig.PreferContiguousImageBuffers = true;
            using (fullscreen.Lock())
            {
              
            }
            _weakReference=new WeakReference(fullscreen);
        }

        return bitmaps;
    }
    public static WeakReference Dispose()
    {
        screenCaptureService.Dispose();
        screenCaptureService = null;
        return _weakReference;
    }
}

In Another Class:

 var captureAllScreen = CaptureAllScreen();
        var weakReference = Dispose();
        while (weakReference.IsAlive)
        {
            GC.Collect();
            GC.WaitForPendingFinalizers();
            Thread.Sleep(100);
        }

[DarthAffe]

ohh nice catch! it's actually leaking textures if a screen-capture is disposed without unregistering all zones frist.
i'll fix that - it's not a big thing.

btw. be carefull: in your code you can leak a the screenCapture if you call your method twice without disposing first, and you can also leak the bitmaps.

If you really just want to take a screenshot of all displays without keeping the screencapture open for performance reasons i'd go with something like this:

IEnumerable<Bitmap> CaptureAllScreen()
{
    using IScreenCaptureService screenCaptureService = new DX11ScreenCaptureService();
    IEnumerable<GraphicsCard> graphicsCards = screenCaptureService.GetGraphicsCards();

    foreach (Display display in screenCaptureService.GetDisplays(graphicsCards.First()))
    {
        using IScreenCapture screenCapture = screenCaptureService.GetScreenCapture(display);
        ICaptureZone captureZone = screenCapture.RegisterCaptureZone(0, 0, display.Width, display.Height);

        while (!screenCapture.CaptureScreen()) ;

        yield return captureZone.Image.ToBitmap();
    }
}

(the ToBitmap is just a extension i use to create a bitmap - replace it with whatever conversion you need).

if you want to keep the ScreenCaptureService alive, i'd not create it in the capture method but in a static constructor or an initialize method instead.