kokorin / Jaffree

______ Stop the War in Ukraine! _______ Java ffmpeg and ffprobe command-line wrapper
Apache License 2.0
471 stars 80 forks source link

Small refactor on ProcessHandler.interactWithProcess() #362

Closed marklassau closed 1 year ago

marklassau commented 1 year ago

Just a suggestion.

The status is always set exactly once - so the null value in the current code is never used.

Setting the variable to final ensures the invariant that it is always set exactly one time - else there will be a compiler error.