https://bugs.kde.org/show_bug.cgi?id=402369
--- Comment #6 from Philippe Waroquiers <philippe.waroqui...@skynet.be> --- (In reply to Nick Nethercote from comment #5) > > It might be interesting to replace the wordFM by an xtree, > > It may. Nonetheless, I'd rather land the code as-is, because it's a major > improvement over the existing DHAT. We can consider optimizations to the > implementation later :) Optimization (in terms of speed) is one (non major) aspect (I would guess that it is unlikely that dhat spends a lot of cpu in this data structure). The main aspect is to avoid adding another dhat specific data structure, instead of reusing (extending) the 'common coregrind data structure', used by massif/memcheck/helgrind. Or told otherwise: avoid growing the code basis unnecessarily. But I have not looked much in details how easy it would be to extend xtre to make it support the dhat and output json. -- You are receiving this mail because: You are watching all bug changes.
