Skip to content

Fixed DirectShow video capture issues.#3298

Merged
opencv-pushbot merged 3 commits intoopencv:2.4from
a-wi:DShow_capture
Oct 9, 2014
Merged

Fixed DirectShow video capture issues.#3298
opencv-pushbot merged 3 commits intoopencv:2.4from
a-wi:DShow_capture

Conversation

@a-wi
Copy link
Copy Markdown

@a-wi a-wi commented Oct 2, 2014

  1. Set correct value of sample size when calling IAMStreamConfig::SetFormat function. For non-RGB media type it can be set to zero value.
    (See http://msdn.microsoft.com/en-us/library/windows/desktop/dd373477%28v=vs.85%29.aspx)
    This fixes a bug reported in http://code.opencv.org/issues/3864.
  2. Fixed media type GUID for I420 format. FCC code bytes should be placed in the reverse order.
  3. Fixed getting proper FCC property value.

@a-wi a-wi force-pushed the DShow_capture branch 2 times, most recently from e2f6c5c to 2d92573 Compare October 2, 2014 18:06
Save selected FCC in order to return proper CV_CAP_PROP_FOURCC property value (currently FCC for RGB24 is always returned).
@vpisarev vpisarev self-assigned this Oct 9, 2014
@vpisarev
Copy link
Copy Markdown
Contributor

vpisarev commented Oct 9, 2014

👍

@opencv-pushbot opencv-pushbot merged commit 5c96021 into opencv:2.4 Oct 9, 2014
vpisarev added a commit that referenced this pull request Oct 9, 2014
@a-wi a-wi deleted the DShow_capture branch November 11, 2014 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

port/backport done Label for maintainers. Authors of PR can ignore this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants