change magic num to macro