Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update PushSourceDesktop.cpp #42

Closed
wants to merge 1 commit into from
Closed

Update PushSourceDesktop.cpp #42

wants to merge 1 commit into from

Conversation

mca64
Copy link

@mca64 mca64 commented Jul 30, 2014

client window --> whole window

but good to have both

client window --> whole window
@rdp
Copy link
Owner

rdp commented Aug 1, 2014

Was this tested then?

On Wed, Jul 30, 2014 at 1:25 PM, mca64 notifications@github.com wrote:

client window --> whole window

but good to have both

You can merge this Pull Request by running

git pull https://github.com/mca64/screen-capture-recorder-to-video-windows-free patch-1

Or view, comment on, or merge it at:

#42
Commit Summary

  • Update PushSourceDesktop.cpp

File Changes

Patch Links:

https://github.com/rdp/screen-capture-recorder-to-video-windows-free/pull/42.patch

https://github.com/rdp/screen-capture-recorder-to-video-windows-free/pull/42.diff


Reply to this email directly or view it on GitHub
#42
.

@mca64
Copy link
Author

mca64 commented Aug 1, 2014

i didnt compile it, but this changes i use for screenshot in delphi to get client or full window. Here you can see (client for StarCraft, full window for rest active app): http://pastebin.com/vkrPNgnz

for client:
DC := GetDC(Okno); Windows.GetClientRect(Okno, Wspolrzedne);

for full window:
DC := GetWindowDC(Okno); Windows.GetWindowRect(Okno, Wspolrzedne);

@rdp
Copy link
Owner

rdp commented Aug 25, 2014

OK I think we got this one mostly :)

@rdp rdp closed this Aug 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants