smasherprog / screen_capture_lite

cross platform screen/window capturing library
MIT License
638 stars 156 forks source link

bug,crash #128

Closed introspection3 closed 2 years ago

introspection3 commented 2 years ago

image

introspection3 commented 2 years ago

It is not recommended to add an assertion in the middle of the function, it is usually an exception image

smasherprog commented 2 years ago

Were you able to figure out the bug? Pull requests are welcome. From the screenshot, there isnt enough information.

introspection3 commented 2 years ago

@smasherprog are you oline? you can download a aspia client,i will give you an account ,you can remote control my pc

introspection3 commented 2 years ago

@smasherprog I have two monitor

smasherprog commented 2 years ago

I did re review your snapshot. In the demo code you are running the window capturing is testing. If you dont have a window matching the name in the code, the application will crash. Try changing the test to find a window that you have open to capture.

smasherprog commented 2 years ago

I just updated master with some code that should help. Give it a shot!

introspection3 commented 2 years ago

@smasherprog may be because :I had adjusted the two monitors's default order

introspection3 commented 2 years ago

image

smasherprog commented 2 years ago

Try taking a screenshot without the assert pop up in the way. I want to see the monitor dimensions listed in the window.

introspection3 commented 2 years ago

image

introspection3 commented 2 years ago

@smasherprog

smasherprog commented 2 years ago

I havent had time to look into this just yet but I will.

smasherprog commented 2 years ago

I am interested to see what the monitor values are contained in it. Could you get those values? base.Imagebuffereize monheight srcrowstride dstrowstride

These values in the image image

introspection3 commented 2 years ago

dear sir ,you are too lazy as I'm here waiting for you three days..... I have searched solution but not found it,you can give me a exe with log

smasherprog commented 2 years ago

I did just push an update to change the assert. It might fix it, since I cannot reproduce your problem

introspection3 commented 2 years ago

@smasherprog good job! sir how to resolve it?

introspection3 commented 2 years ago

with this lib,I think you should have a new remote desktop software

smasherprog commented 2 years ago

glad to hear its fixed!