I think you're right. it should be as well. I'll take a look tonight
when I get home.

peter


On 8/16/05, sebb <[EMAIL PROTECTED]> wrote:
> Should that have been
> 
>        res.setContentLength(...getContentLength())
> 
> instead? Or as well?
> 
> S.
> On 16 Aug 2005 01:29:51 -0000, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
> > woolfel     2005/08/15 18:29:51
> >
> >  Modified:    src/protocol/jms/org/apache/jmeter/protocol/jms/sampler
> >                        JMSSampler.java
> >  Log:
> >  reverted the change, since it causes a compilation error.
> >  peter
> >
> >  Revision  Changes    Path
> >  1.12      +3 -3      
> > jakarta-jmeter/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java
> >
> >  Index: JMSSampler.java
> >  ===================================================================
> >  RCS file: 
> > /home/cvs/jakarta-jmeter/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/JMSSampler.java,v
> >  retrieving revision 1.11
> >  retrieving revision 1.12
> >  diff -u -r1.11 -r1.12
> >  --- JMSSampler.java   15 Aug 2005 20:33:07 -0000      1.11
> >  +++ JMSSampler.java   16 Aug 2005 01:29:51 -0000      1.12
> >  @@ -139,9 +139,9 @@
> >                                        }
> >                                } else {
> >                                        if (replyMsg instanceof TextMessage) 
> > {
> >  -                                             
> > res.setResponseData(((TextMessage) replyMsg).getText().getContentLength());
> >  +                                             
> > res.setResponseData(((TextMessage) replyMsg).getText().getBytes());
> >                                        } else {
> >  -                                             
> > res.setResponseData(replyMsg.toString().getContentLength());
> >  +                                             
> > res.setResponseData(replyMsg.toString().getBytes());
> >                                        }
> >                                        res.setSuccessful(true);
> >                                }
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to