making proc private instead of public