OK so here's another patch that logs a message if the command isn't found.
Anyway, if you think that the try/catch clause is better/enough then let it be.
CommandNotificationHandlerImpl.java.diff (620 Bytes)
--- On Fri, 3/15/13, Lyubomir Marinov <firstname.lastname@example.org> wrote:
From: Lyubomir Marinov <email@example.com>
Subject: [jitsi-dev] Re: PATCH avoid exception if command file not found
Date: Friday, March 15, 2013, 6:16 AM
2013/3/15 Vieri <firstname.lastname@example.org>:
> Simple patch to avoid exception when command file not
Your patch may prevent the IOException but it also fails to
error/warning message that the command file does not exist.
> I still don't know why the try/catch doesn't avoid it
but anyway this patch seems to solve the issue.
The catch does not "avoid" the IOException but it handles it
an error message and the stack trace. Moreover, the
message contains additional information about the cause.
Overall, I think the patch in its current form should not be