Hello, This looks good to me, except for some nit picking: sprintf could be snprintf and probably you only need one 'associative'. However I would be happy for this to be merged as-is. -- Thank you, Richard.