base_object::check_collision() should take an object pointer.

Bug #1272733 reported by Rick Stovall
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
New Real Time Battle
Status tracked in Alpha
Alpha
Fix Released
Critical
Rick Stovall

Bug Description

in /cpp/sim_engine/base_object/base_object.[h|cpp] :

bool base_object::check_collision(sphere s)
{
  float r = s.radius + bounding_sphere.radius;
  return r <=
    s.center.range(bounding_sphere.center+location);
};

This method should take an object_p (a pointer to an object) instead of a sphere; this will greatly simplify collision checks.

Related branches

Rick Stovall (fpstovall)
Changed in nrtb:
importance: Undecided → Critical
status: New → In Progress
assignee: nobody → Rick Stovall (fpstovall)
milestone: none → sprint-005a
Revision history for this message
Rick Stovall (fpstovall) wrote :

Fixed... found an additional issue where the method was returning an inverted truth table (discovered during unit test).

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.