Здравствуйте, malegkin, Вы писали:

M>Давненько я не брал в руки плюсы...
M>Буду признателен за критику кода решения задачки http://acm.timus.ru/problem.aspx?space=1&num=1712
M>Волнует в первую очередь move semantics / передача и возврат объектов между функциями.
M>Возможно ли repeat сделать проще? ну лень мне каждый раз писать for(;) ))

Мои 5 копеек — inline не нужен практически всегда. Единственное его применение — определение функции в заголовочных файлах, чтобы предотвратить ошибки линкора о множественных именах для такой функции (а также inline variable начиная с C++17).

Еще вместо std::string для строк square_t можно сделать двумерный массив символов, но это вопрос вкуса.

И repeat можно сделать проще, написав обычный цикл for, знакомый 100% читателей