jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Philippe Mouawad <philippe.moua...@gmail.com>
Subject Re: svn commit: r1737490 - in /jmeter/trunk: src/core/org/apache/jmeter/services/FileServer.java test/src/org/apache/jmeter/config/TestCVSDataSet.java test/src/org/apache/jmeter/services/TestFileServer.java
Date Sat, 02 Apr 2016 12:44:02 GMT
Ignore my note, I mixed up.
Thanks

On Sat, Apr 2, 2016 at 2:23 PM, Philippe Mouawad <philippe.mouawad@gmail.com
> wrote:

> I didn't follow this PR closely, but it is correct to remove this code:
> if (fileEntry.exception != null) {
>             Throwable exception = fileEntry.exception;
>             if (exception instanceof RuntimeException) {
>                 throw (RuntimeException) exception;
>             }
>             // Note: previous exception message should be clear enough, so
>             // no additional information like "file name" needs to be
> added here
>             throw new IllegalStateException(exception.getMessage(),
> exception);
>         }
>
> Thanks
>
> On Sat, Apr 2, 2016 at 2:17 PM, Vladimir Sitnikov <
> sitnikov.vladimir@gmail.com> wrote:
>
>> Philippe> I see this part of code has been dropped at line 290, is
>> Philippe> this regular ?:
>>
>> Can you clarify the question?
>>
>> Vladimir
>>
>
>
>
> --
> Cordialement.
> Philippe Mouawad.
>
>
>


-- 
Cordialement.
Philippe Mouawad.

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message