search

jackburton79 / bescreencapture

BeScreenCapture - Screen Capture software for Haiku
BSD 3-Clause "New" or "Revised" License
7 stars 10 forks source link

Can set an invalid capture area via hey #56

Closed jackburton79 closed 1 year ago

jackburton79 commented 1 year ago

Using hey is possible to set an invalid capture area. For example:

~> hey BeScreenCapture SET CaptureRect to "BRect(-1,-1,99999,9999)" Reply BMessage(B_REPLY): "error" (B_INT32_TYPE) : 0 (0x00000000)

the capture rect is set, but

This error is logged on console: bitmap bounds is much too large: BRect(-1.0, -1.0, 99999.0, 9999.0)

jackburton79 commented 1 year ago

Fixed. Now Controller::SetCaptureArea() checks if the rect is bigger than screen.