You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2017/03/18 12:18:12 UTC

svn commit: r1787551 - /jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java

Author: pmouawad
Date: Sat Mar 18 12:18:12 2017
New Revision: 1787551

URL: http://svn.apache.org/viewvc?rev=1787551&view=rev
Log:
Log exception

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java?rev=1787551&r1=1787550&r2=1787551&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java Sat Mar 18 12:18:12 2017
@@ -583,7 +583,8 @@ public class SampleSaveConfiguration imp
             }
             return temporaryTimestampFormat;
         } catch(IllegalArgumentException ex) {
-            log.error("Invalid pattern value {} for property {}", temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP);
+            log.error("Invalid pattern value {} for property {}", temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP, 
+                    ex);
             return null;
         }
     }



Re: svn commit: r1787551 - /jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java

Posted by Felix Schumacher <fe...@internetallee.de>.
Am 18.03.2017 um 20:12 schrieb Antonio Gomes Rodrigues:
> Hi,
>
> How do you see it?
>
> In Linux, I can't see this character
>
> less src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java  |
> grep temporaryTimestampFormat | grep TIME_STAMP_FORMAT_PROP
>          String temporaryTimestampFormat =
> props.getProperty(TIME_STAMP_FORMAT_PROP, MILLISECONDS);
>                          temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP);
>              log.error("Invalid pattern value {} for property {}",
> temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP,
I use vim with something like this in my .vimrc:

" Show trailing whitespace visually
  if (&termencoding == "utf-8") || has("gui_running")
     if v:version >= 700
        set list listchars=tab:��,trail:�,extends:\u2026,nbsp:\u2017
     else
        set list listchars=tab:��,trail:�,extends:\u2026
     endif
  else
     if v:version >= 700
        set list listchars=tab:>-,trail:.,extends:>,nbsp:_
     else
        set list listchars=tab:>-,trail:.,extends:>
     endif
  endif

The character was a no-break space unicode character U+00A0.

Regards,
  Felix
>
>
> Antonio
>
> 2017-03-18 19:38 GMT+01:00 Philippe Mouawad <ph...@gmail.com>:
>
>> Invisible Mac char.
>> Good catch, I'll fix it tomorrow or if you want go ahead
>> Thanks
>>
>> On Saturday, March 18, 2017, sebb <se...@gmail.com> wrote:
>>
>>> On 18 March 2017 at 12:18,  <pmouawad@apache.org <javascript:;>> wrote:
>>>> Author: pmouawad
>>>> Date: Sat Mar 18 12:18:12 2017
>>>> New Revision: 1787551
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=1787551&view=rev
>>>> Log:
>>>> Log exception
>>>>
>>>> Modified:
>>>>      jmeter/trunk/src/core/org/apache/jmeter/samplers/
>>> SampleSaveConfiguration.java
>>>> Modified: jmeter/trunk/src/core/org/apache/jmeter/samplers/
>>> SampleSaveConfiguration.java
>>>> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
>>> apache/jmeter/samplers/SampleSaveConfiguration.java?
>>> rev=1787551&r1=1787550&r2=1787551&view=diff
>>>> ============================================================
>>> ==================
>>>> --- jmeter/trunk/src/core/org/apache/jmeter/samplers/
>> SampleSaveConfiguration.java
>>> (original)
>>>> +++ jmeter/trunk/src/core/org/apache/jmeter/samplers/
>> SampleSaveConfiguration.java
>>> Sat Mar 18 12:18:12 2017
>>>> @@ -583,7 +583,8 @@ public class SampleSaveConfiguration imp
>>>>               }
>>>>               return temporaryTimestampFormat;
>>>>           } catch(IllegalArgumentException ex) {
>>>> -            log.error("Invalid pattern value {}� for property {}",
>>> temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP);
>>>> +            log.error("Invalid pattern value {}� for property {}",
>>> temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP,
>>>
>>> What's the character after {} doing?
>>>
>>>> +                    ex);
>>>>               return null;
>>>>           }
>>>>       }
>>>>
>>>>
>>
>> --
>> Cordialement.
>> Philippe Mouawad.
>>


Re: svn commit: r1787551 - /jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java

Posted by Antonio Gomes Rodrigues <ra...@gmail.com>.
Hi,

How do you see it?

In Linux, I can't see this character

less src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java  |
grep temporaryTimestampFormat | grep TIME_STAMP_FORMAT_PROP
        String temporaryTimestampFormat =
props.getProperty(TIME_STAMP_FORMAT_PROP, MILLISECONDS);
                        temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP);
            log.error("Invalid pattern value {} for property {}",
temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP,


Antonio

2017-03-18 19:38 GMT+01:00 Philippe Mouawad <ph...@gmail.com>:

> Invisible Mac char.
> Good catch, I'll fix it tomorrow or if you want go ahead
> Thanks
>
> On Saturday, March 18, 2017, sebb <se...@gmail.com> wrote:
>
> > On 18 March 2017 at 12:18,  <pmouawad@apache.org <javascript:;>> wrote:
> > > Author: pmouawad
> > > Date: Sat Mar 18 12:18:12 2017
> > > New Revision: 1787551
> > >
> > > URL: http://svn.apache.org/viewvc?rev=1787551&view=rev
> > > Log:
> > > Log exception
> > >
> > > Modified:
> > >     jmeter/trunk/src/core/org/apache/jmeter/samplers/
> > SampleSaveConfiguration.java
> > >
> > > Modified: jmeter/trunk/src/core/org/apache/jmeter/samplers/
> > SampleSaveConfiguration.java
> > > URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> > apache/jmeter/samplers/SampleSaveConfiguration.java?
> > rev=1787551&r1=1787550&r2=1787551&view=diff
> > > ============================================================
> > ==================
> > > --- jmeter/trunk/src/core/org/apache/jmeter/samplers/
> SampleSaveConfiguration.java
> > (original)
> > > +++ jmeter/trunk/src/core/org/apache/jmeter/samplers/
> SampleSaveConfiguration.java
> > Sat Mar 18 12:18:12 2017
> > > @@ -583,7 +583,8 @@ public class SampleSaveConfiguration imp
> > >              }
> > >              return temporaryTimestampFormat;
> > >          } catch(IllegalArgumentException ex) {
> > > -            log.error("Invalid pattern value {}Â for property {}",
> > temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP);
> > > +            log.error("Invalid pattern value {}Â for property {}",
> > temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP,
> >
> > What's the character after {} doing?
> >
> > > +                    ex);
> > >              return null;
> > >          }
> > >      }
> > >
> > >
> >
>
>
> --
> Cordialement.
> Philippe Mouawad.
>

Re: svn commit: r1787551 - /jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java

Posted by Philippe Mouawad <ph...@gmail.com>.
Invisible Mac char.
Good catch, I'll fix it tomorrow or if you want go ahead
Thanks

On Saturday, March 18, 2017, sebb <se...@gmail.com> wrote:

> On 18 March 2017 at 12:18,  <pmouawad@apache.org <javascript:;>> wrote:
> > Author: pmouawad
> > Date: Sat Mar 18 12:18:12 2017
> > New Revision: 1787551
> >
> > URL: http://svn.apache.org/viewvc?rev=1787551&view=rev
> > Log:
> > Log exception
> >
> > Modified:
> >     jmeter/trunk/src/core/org/apache/jmeter/samplers/
> SampleSaveConfiguration.java
> >
> > Modified: jmeter/trunk/src/core/org/apache/jmeter/samplers/
> SampleSaveConfiguration.java
> > URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/samplers/SampleSaveConfiguration.java?
> rev=1787551&r1=1787550&r2=1787551&view=diff
> > ============================================================
> ==================
> > --- jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java
> (original)
> > +++ jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java
> Sat Mar 18 12:18:12 2017
> > @@ -583,7 +583,8 @@ public class SampleSaveConfiguration imp
> >              }
> >              return temporaryTimestampFormat;
> >          } catch(IllegalArgumentException ex) {
> > -            log.error("Invalid pattern value {}Â for property {}",
> temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP);
> > +            log.error("Invalid pattern value {}Â for property {}",
> temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP,
>
> What's the character after {} doing?
>
> > +                    ex);
> >              return null;
> >          }
> >      }
> >
> >
>


-- 
Cordialement.
Philippe Mouawad.

Re: svn commit: r1787551 - /jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java

Posted by sebb <se...@gmail.com>.
On 18 March 2017 at 12:18,  <pm...@apache.org> wrote:
> Author: pmouawad
> Date: Sat Mar 18 12:18:12 2017
> New Revision: 1787551
>
> URL: http://svn.apache.org/viewvc?rev=1787551&view=rev
> Log:
> Log exception
>
> Modified:
>     jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java?rev=1787551&r1=1787550&r2=1787551&view=diff
> ==============================================================================
> --- jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java Sat Mar 18 12:18:12 2017
> @@ -583,7 +583,8 @@ public class SampleSaveConfiguration imp
>              }
>              return temporaryTimestampFormat;
>          } catch(IllegalArgumentException ex) {
> -            log.error("Invalid pattern value {}Â for property {}", temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP);
> +            log.error("Invalid pattern value {}Â for property {}", temporaryTimestampFormat, TIME_STAMP_FORMAT_PROP,

What's the character after {} doing?

> +                    ex);
>              return null;
>          }
>      }
>
>