Запутался с define

да действительно туплю с утра

Тело сишного макроса подставляется в исходный код. Таким образом, при вызове макроса

max_value = max(++x, y++);

после обработки препроцессором получим

max_value = ((++x)>(y++) ? (++x) : (y++));

и в зависимости от результата вычисления условного выражения тернарного оператора одна из переменных будет инкрементирована два раза.

Поведение, которое ты ожидаешь, достигается с помощью функций.

И да, с сишными макросами в подобных выражениях есть вероятность наткнуться на нарушение точек следования (не в данном примере), так что надо быть осторожней с побочными эффектами ☺

мне просто было интересно чего оно так, угу я сделал что-то типа

template <class T>
inline const T& max(const T& a, const T& b) 
{
	return a > b ? a : b;
}

max уже придумали за тебя