OK I'm a jerk...
I was testing List Casting Spell on an Hybrid Class, without even looking at the returned count "0"...
When testing on a list caster, the spell list also trigger a memory leak, so that explains a lot of weird things I saw in profiler
So the definitive leaker is my new method "SkillBase.GetSpellList(string line)", which is trying to clone each Spell object before sending it (preventing from modifying SkillBase objects !!), and this leads me to the new "Tooltip" code within Spell, which is probably triggering something as it is called by the Spell constructor on Clone()'ing...
I think this whole tooltip thing handled in spell object is pretty bad, for all other skills I rely on a database field and it already have some flows :
GameEffect, like Berserk triggered by an Ability, doesn't have a TooltipID when it should...
Abilities should trigger SpellEffect everytime ! I can't use an Ability ID to target an Ability Effect, client expect a spell ID for buff/debuff display icons.
We shouldn't be handling ANYTHING else than GameSpellEffect (to display in player UI...)
So I'm not gonna rely on weird code to support tooltip just for spell when there is nothing around fixing berserk or such effects, we don't have a portal working with 1.115+ client anyway
Shards Admin / DOL User will need to update database records to have a flawless 1.115+ experience...