Skip to content

dshow: Add ability to save directshow graph#15467

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
elms:dshow/add_save_graph_file
Oct 2, 2019
Merged

dshow: Add ability to save directshow graph#15467
opencv-pushbot merged 1 commit intoopencv:3.4from
elms:dshow/add_save_graph_file

Conversation

@elms
Copy link
Copy Markdown
Contributor

@elms elms commented Sep 5, 2019

This pullrequest changes

Adds ability to save the directshow graph by setting an environment variable providing a filename.

force_builders=win32

@elms elms force-pushed the dshow/add_save_graph_file branch 2 times, most recently from aeeec9f to f88d035 Compare September 6, 2019 13:42
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for contribution!

This patch should go into 3.4 branch first. We will merge changes from 3.4 into master regularly (weekly/bi-weekly).

So, please:

  • change "base" branch of this PR: master => 3.4 (use "Edit" button near PR title)
  • rebase your commits from master onto 3.4 branch. For example:
    git rebase -i --onto upstream/3.4 upstream/master
    (check list of your commits, save and quit (Esc + "wq" + Enter)
    where upstream is configured by following this GitHub guide and fetched (git fetch upstream).
  • push rebased commits into source branch of your fork (with --force option)

Note: no needs to re-open PR, apply changes "inplace".

@elms elms changed the base branch from master to 3.4 September 6, 2019 17:33
@elms elms force-pushed the dshow/add_save_graph_file branch from f88d035 to aeb5c5d Compare September 6, 2019 17:34
@elms
Copy link
Copy Markdown
Contributor Author

elms commented Sep 6, 2019

Rebased to 3.4

@elms elms requested a review from alalek September 9, 2019 12:30
Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done!

Copy link
Copy Markdown
Contributor

@mshabunin mshabunin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the code according to the comments.

@elms elms force-pushed the dshow/add_save_graph_file branch from aeb5c5d to 8b483a9 Compare October 1, 2019 18:24
@elms elms requested a review from mshabunin October 1, 2019 19:03
@opencv-pushbot opencv-pushbot merged commit 8b483a9 into opencv:3.4 Oct 2, 2019
@alalek alalek mentioned this pull request Oct 2, 2019
@elms elms deleted the dshow/add_save_graph_file branch October 3, 2019 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants