Strip high number of BGP Communities: OpenBGPD example is different in content

[marek22k]

Hello, the OpenBGPD example is different from the other two. Is there a reason for that? Which one is correct?
I have highlighted the difference in pseudo code below.

BIRD, Junos, Arista

if ( ( bgp_community.length + bgp_ext_community.length + bgp_large_community.length ) >= 100 ) {
    bgp_community.delete_all();
    bgp_ext_community.delete_all();
    bgp_large_community.delete_all();
}

OpenBGPD:

if ( bgp_community.length >= 100 ) {
    bgp_community.delete_all();
}
if ( bgp_ext_community.length >= 100 ) {
    bgp_community.delete_all();
}
if ( bgp_large_community.length >= 100 ) {
    bgp_community.delete_all();
}

Ping on maintainers: @job @rodecker

[job]

OpenBGPd doesn’t support cumulatively counting the instances of extended + classic + large communities. I think the goal is to impose some kind of upper bound, not necessarily being precise.