Hi Gary,

I've just fixed the javadoc:
- 
http://svn.apache.org/viewvc/commons/proper/exec/trunk/src/main/java/org/apache/commons/exec/ExecuteWatchdog.java?r1=1741116&r2=1741115&pathrev=1741116

Thanks for the suggestion! :-)

Woonsan


On Tue, Apr 26, 2016 at 5:23 PM, Gary Gregory <garydgreg...@gmail.com> wrote:
> Woonsan,
>
> Would you mind updating the Javadoc?
>
> Thank you,
> Gary
> On Apr 26, 2016 1:32 PM, "Woonsan Ko" <woon...@apache.org> wrote:
>
>> The javadoc seems to have outdated info. It should have been like the
>> following:
>>
>>   ExecuteWatchdog watchdog = new ExecuteWatchdog(30000);
>>   Executor exec = new DefaultExecutor();
>>   exec.setStreamHandler(myStreamHandler);
>>   exec.setWatchdog(watchdog);
>>   int exitvalue = exec.execute(mycmdline);
>>   if (DefaultExecutor.isFailure(exitvalue) && watchdog.killedProcess()) {
>>     // it was killed on purpose by the watchdog
>>   }
>>
>> Regards,
>>
>> Woonsan
>>
>>
>> On Tue, Apr 26, 2016 at 8:32 AM, Leonardo K. Shikida <shik...@gmail.com>
>> wrote:
>> > Hi
>> >
>> >
>> https://commons.apache.org/proper/commons-exec/apidocs/org/apache/commons/exec/ExecuteWatchdog.html
>> >
>> > says
>> >
>> >  ExecuteWatchdog watchdog = new ExecuteWatchdog(30000);
>> >  Executer exec = new Executer(myloghandler, watchdog);
>> >  exec.setCommandLine(mycmdline);
>> >  int exitvalue = exec.execute();
>> >  if (Execute.isFailure(exitvalue) && watchdog.killedProcess()) {
>> >      // it was killed on purpose by the watchdog
>> >  }
>> >
>> > but
>> >
>> > "Executer" does not exist
>> >
>> > "Executor" does exist but it is an interface
>> >
>> > DefaultExecutor implements Executor but does not set a watchdog in the
>> > constructor
>> >
>> >
>> >
>> >
>> > []
>> >
>> > Leo
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: user-unsubscr...@commons.apache.org
>> For additional commands, e-mail: user-h...@commons.apache.org
>>
>>

---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscr...@commons.apache.org
For additional commands, e-mail: user-h...@commons.apache.org

Reply via email to