Make "Trim flag" an API function

From what I gather from patch you are just unflagging flags beyond cutoff point. This is not a real trim in the sense that it won't actually delete the flag record from the database. Am I correct? If this is correct then the flag entries will just continue to grow and may not benefit in the long term for performance and storage. This is not worse than the current but loosing the potential benefits possible. Shouldn't we actuall delete the records?

SOrry for asking stupid question. I get that whatever was part of Rules action is now moved to API. But I still don't get where we actually delete the record? I looked at the patch above and as part of trim you are just unflagging. Does this mean the rules will be deleting the records after call to this API? Would n't be nice this is available as 6.x core action so there is no need for rules module?

Ok I see what you are doing. Just refactoring and moving code from "flag_rules_action_trim" to another seperate function flag_trim_flag.

But what is real value addition of original flag_rules_action_trim as all it is doing is calling flag_trim_flag? Sorry if I am asking too many dumb questions.

Committed to the 2.x branch (not sure why I can't get this tagged as 2.x yet since the branch exists).