On Mon, 2024-11-25 at 16:06 -0500, Benjamin Marzinski wrote: > On Mon, Nov 25, 2024 at 03:32:23PM +0100, Martin Wilck wrote: > > pathcmp() makes sure that all paths in pgp have a match in cpgp, > > but not > > vice-versa. Check the number of paths, too. > > This looks fine. But looking at it made we a nervous about cpgp->id. > We > only calculate that in pgcmp() (and just for mpp, not cmpp) and in > disassemble_map(). But we clearly can have pathgroup changes after > that, > as the last patch has shown. To be safe, we should either skip the > whole > pgp->id thing (it's not a huge time savings) or recalculate it for > all > of cmpp's path groups before we start the loops in pgcmp(). Agreed. Like before, I am wondering if we should drop pgp->id in 0.11.0 already or postpone. IMO my patch is a valid fix for a minor issue which needs redesign in a future release. Martin
