title: Memory Leak from iterating Opencv frames
body: <p>I am using the java wrapper of OpenCV. I tried to write an Iterator over frames of a film. My problem is that the iterator is a huge memory leak. Here is a very simplified version of the iterator, which has this leak:</p>
  
  <pre><code>public static final class SimpleIt implements Iterator&lt;Mat&gt; {
  
      private final VideoCapture capture;
      boolean hasNext;
  
      public SimpleIt(final VideoCapture capture) {
          this.capture = capture;
          hasNext = capture.grab();
      }
  
      @Override
      public boolean hasNext() {
          return hasNext;
      }
  
      @Override
      public Mat next() {
          final Mat mat = new Mat();
          capture.retrieve(mat);
          hasNext = capture.grab();
          return mat;
      }
  }
  </code></pre>
  
  <p>I Iterate over this code using this loop:</p>
  
  <pre><code>    final VideoCapture vc = new VideoCapture("/path/to/file");
      final SimpleIt it = new SimpleIt(vc);
      while (it.hasNext) {
          it.next();
      }
  </code></pre>
  
  <p>Just iterating will increase memory consumption linear. I see that the problem is the first line in the next()-Method. It always creates a new Mat. But speaking of java alone, this Mat will run out of scope as soon as the iterating code iterates to the next image.</p>
  
  <p>I could overcome the problem, by not using a new Mat every time, but overwriting always the same Mat-Object, like this:</p>
  
  <pre><code>    private final VideoCapture capture;
      private final Mat mat = new Mat();
      boolean hasNext;
  
      @Override
      public Mat next() {
          capture.retrieve(mat);
          hasNext = capture.grab();
          return mat;
      }
  </code></pre>
  
  <p>But now the last frame which was given by the iterator will be overwritten. Thus, I cannot hold it outside for later use, if I am interested in this single frame. I could copy it, of course, but that would also be expensive.</p>
  
  <p>I assume that the problem is that the garbage collector will not destroy the Mat objects, because it does not recognize the memory consumption, since it is not java heap space. Calling mat.release() in the loop will help, but of course in real code this means I will have no garbage collection for my Mat objects.</p>
  
  <p>Anybody has an idea how to do it?</p>
  
  <p>Edit:</p>
  
  <p>Since it seems not to be clear what the problem with my second solution is, I write it down more explicitly. Think about the following code, using the iterator:</p>
  
  <pre><code>    final VideoCapture vc = new VideoCapture("/path/to/file");
      final SimpleIt it = new SimpleIt(vc);
      int i = 0;
      Mat save = null;
      while (it.hasNext) {
          final Mat next = it.next();
          if (i == 10) {
              save = next;
              Highgui.imwrite("/path/to/10.png", save);
          } else if (i == 30) {
              Highgui.imwrite("/path/to/30.png", save);
          }
          i++;
      }
  </code></pre>
  
  <p>With the second version of the iterator, 10.png, and 30.png will be different images. But that's obviously not what was intended.</p>