savecore: Complete libxo transition.

[des]

sbin/savecore/savecore.c

1568

[des]

sbin/savecore/savecore.c

1551

strange, why not logmsg() here? are there other instances?

[bses30074_gmail.com]

sbin/savecore/savecore.c

1551

I scan the code quickly. It seems if the 'checkfor' flag is true, it uses the printf function instead of logmsg function. The checkfor flag is set if -C is set. The man page of savecore says "display a brief message to indicate the status". So maybe the brief message means using the simple printf function to show the message.

[bses30074_gmail.com]

Update with exit(EXIT_SUCCESS) to replace return (0)

[bses30074_gmail.com]

sbin/savecore/savecore.c

1568

Ask about this code. Should I always using exit function instead of return in main function.

[des]

sbin/savecore/savecore.c

1568

It's a matter of taste. The result is the same but calling exit() makes it clearer what's going on. It's especially relevant in programs that have multiple exit points, as is the case here.