*:int Insert(const char*, bool check_for_dups=true)
*:int Insert(ExprTree*, bool check_for_dups=true)
-New {quote:ClassAds} already have a compatibility version of the first variant. But it doesn't handle string escaping properly.
+New {quote:ClassAds} already have a compatibility version of the first variant. But it doesn't handle string escaping properly. It also appears to leak memory.
For the second variant, we'd probably use new {quote:ClassAds}' Insert(), which takes the attribute name and value as separate arguments. The ExprTree's passed in as arguments come from calls to NextExpr(), Lookup(), or Parse() near the callsite. These should be straightforward to update.
