[Scummvm-tracker] [ScummVM :: Bugs] #16370: BAGEL: HODJNPODJ: Art Parts seems broken when moving multiple pieces at once

[ScummVM :: Bugs]

#16370: BAGEL: HODJNPODJ: Art Parts seems broken when moving multiple pieces at
once
-------------------------+----------------------------
Reporter:  eriktorbjorn  |       Owner:  dreammaster
    Type:  defect        |      Status:  closed
Priority:  normal        |   Component:  Engine: BAGEL
 Version:                |  Resolution:  fixed
Keywords:                |        Game:  Hodj'n'Podj
-------------------------+----------------------------
Changes (by dreammaster):

 * resolution:   => fixed
 * status:  new => closed

Comment:

 Found the cause. Sigh.. turns out the implementation of the
 CRect::IntersectRect was wrong. I was returning whether the two passed
 rects overlapped, when but they did, it wasn't actually setting the rect
 fields to the overlap area. So when it tries to use that to figure out
 which tiles go where, it screws up the calculations and ends up with the
 duplication you saw.

 I guess I should be grateful. As a core class, the CRect IntersectRect is
 used in a variety of places, including Archeroids, so identifying and
 fixing it may have helped avoid a myriad of future weird bug reports.
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/16370#comment:5>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM