clang-tidy: replace macro with enum (modernize-macro-to-enum)

clang-tidy: macro 'MAX_FISH' defines an integral constant; prefer an enum instead (modernize-macro-to-enum)

clang-tidy: the parameter 'f_left' is copied for each invocation but only used as a const reference; consider making it a const reference (performance-unnecessary-value-param)

clang-tidy: parameter 'f_right' is passed by value and only copied once; consider moving it to avoid unnecessary copies (performance-unnecessary-value-param)

Found 4 clang-tidy issues

The following actions uses node12 which is deprecated and will be forced to run on node16: asarium/clang-tidy-action@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/