Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#217 closed defect (fixed)

cropdetect filter over-rounding down width and height.

Reported by: willo@… Owned by: alex@…
Priority: normal Component: vf
Version: unspecified Severity: normal
Keywords: Cc:
Blocked By: Blocking:
Reproduced by developer: Analyzed by developer:


CVS version 1.10 of vf_cropdetect.c I believe mis-calculates the initial width
and height values. This was not happening to me before the "round" parameter
and code was added.

Looking at the code, I believe the lines 112 and 113 should add one onto the
difference between {x2,y2} and {x1,y1}.

As a simple example, assume that I have an NTSC disk that uses the entirity of
the screen. It would have ranges of "X: 0..719 Y:0..479". And the resulting
width and heights *should* be 720 and 480, but the code will generate values of
719 and 479 (before the rounding and trimmming steps, of course.)

I think this fix is compatable with the way the code handles all the other
rounding and shrinking.

Anyway, now I'm in the habit of looking at the raw "X: x1..x2 Y:y1..y2" numbers
and calculating it myself. But I think this should be fixed.

Change History (1)

comment:1 Changed 13 years ago by reimar

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in CVS together with another error...

Note: See TracTickets for help on using tickets.