diff&cmp not suitable to be called as library functions.

Problems with using diffutils.rs as a library:

1. ExitCode is intended to be consumed only by the standard library ... For forwards compatibility with potentially unusual targets, this type currently does not provide Eq, Hash, or access to the raw value. ... so it seems library callers (such as myself) would prefer an i32.
2. The exit(2) is worrying for library callers who don't want to be suddenly killed.

I got the hy3 model to create a fix for this for me. But it lumped it all into one big commit: https://github.com/gmatht/diffutils/commit/4bdd5219adf1485aaf60114ba10adf7e3a26ac7e

Should I make a pull request including two commits:

1. Replace the ExitCode/exit() convention with i32 returns, everywhere except at the very top main function.
2. Add Tests that ensure errors return an appropriate i32 instead of aborting the whole process.

Also should 1./2. be split into updating diff.rs and updating cmp.rs, resulting in 4 commits? If so, main should also should still be updated in each commit so e.g. it doesn't break the build.

Do you prefer that the test or the fix be commited first?

Empieza revisando diff.rs y cmp.rs; después inspecciona la función main de nivel superior y el commit de corrección enlazado para comprobar el comportamiento actual de ExitCode y exit(). Comprueba cómo están organizadas las pruebas y añade cobertura para los retornos de error sin terminación del proceso. Se considera terminado cuando ambos comandos siguen siendo compilables, los llamadores de la biblioteca reciben resultados i32 apropiados y solo la función main de nivel superior gestiona la salida del proceso.