The default implementation of do_thousands_sep
already returns ','
. It looks like you should override do_grouping
instead. do_grouping
returns an empty string by default, which means no grouping. This means groups of three digits each:
string do_grouping() const
{
return "\03";
}