On Sat, 2008-10-25 at 21:43 -0400, Steve Huston wrote: > Hi Alan, > > The changes to SubscriptionManager.cpp involved use of std::map::at(). > That method is not included in RHEL 4's g++, and also not in > Microsoft's lib. I changed it to use find() instead. > > This raises a question that needs some clarification... The comments > on the SubscriptionManager method say that if the entry is not found, > an empty Subscription is returned. However, if at() doesn't find the > key, it throws an exception. Find() creates an empty entry. Which way > should this really be?
Thanks for catching that. On reflection I think it should throw an exception, I'll commit the change (and update the comment) soon as my build is done.
