From 79cde0e8957c8afebbcfa86c0d21292f1c706727 Mon Sep 17 00:00:00 2001 From: dvlierop2 Date: Sun, 7 Sep 2008 19:02:52 +0000 Subject: [PATCH] Remove an obsolete comment --- src/sp-item.cpp | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/sp-item.cpp b/src/sp-item.cpp index 706469d3c..11d572818 100644 --- a/src/sp-item.cpp +++ b/src/sp-item.cpp @@ -931,10 +931,13 @@ boost::optional sp_item_bbox_desktop(SPItem *item, SPItem::BBoxType ty static void sp_item_private_snappoints(SPItem const *item, SnapPointsIter p) { - boost::optional bbox = item->getBounds(sp_item_i2d_affine(item)); - /* Just the corners of the bounding box suffices given that we don't yet - support angled guide lines. */ + /* This will only be called if the derived class doesn't override this. + * see for example sp_genericellipse_snappoints in sp-ellipse.cpp + * We don't know what shape we could be dealing with here, so we'll just + * return the corners of the bounding box */ + boost::optional bbox = item->getBounds(sp_item_i2d_affine(item)); + if (bbox) { NR::Point p1, p2; p1 = bbox->min(); @@ -944,6 +947,7 @@ static void sp_item_private_snappoints(SPItem const *item, SnapPointsIter p) *p = p2; *p = NR::Point(p1[NR::Y], p2[NR::X]); } + } void sp_item_snappoints(SPItem const *item, bool includeItemCenter, SnapPointsIter p) -- 2.30.2