smasherprog / screen_capture_lite

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

Use XGetWMName instead of XFetchName. #48

Closed tarcisioe closed 6 years ago

tarcisioe commented 6 years ago

XFetchName fails to get the window name in some cases, returning an empty string (in my case, of 5 windows it was fetching only one name correctly). Using XGetWMName makes sure the property is correctly fetched and converted to a string, and is encoding-aware.

smasherprog commented 6 years ago

Looks good to me

smasherprog commented 6 years ago

Ill take a closer look at this when I get home

smasherprog commented 6 years ago

Just posted a release on this. Thanks for the PR