Bug report for JMeter [2011/10/16]

2011-10-16 Thread bugzilla
+---+ | Bugzilla Bug ID | | +-+ | | Status: UNC=Unconfirmed NEW=New ASS=Assigned

RE: [VOTE] Apache JMeter TLP

2011-10-16 Thread Noel J. Bergman
+1 --- Noel - To unsubscribe, e-mail: dev-unsubscr...@jakarta.apache.org For additional commands, e-mail: dev-h...@jakarta.apache.org

Re: [JMeter] Issue 51861 - Improve HTTP Request GUI to better show parameters without name (Raw Body)

2011-10-16 Thread Philippe Mouawad
Hello Sebb, I submitted a new version of patch that takes into account the folliwing: - if the raw flag is set, we don't do a merge; otherwise keep current behaviour (I noticed that if there is only one parameter in request with no name and there are parameters in request default, only p

[Bug 52040] Add a toolbar in JMeter main window

2011-10-16 Thread Philippe Mouawad
Hello Milamber, I vote for it, I find it great : - we gain one click each time - I found icons were very meaningful Some notes (matter of taste :-)): - - and + buttons seem bit wide no ? Maybe one missing icon for "Start no pauses". Regards Philippe

Seemingly incorrect short-circuiting logic

2011-10-16 Thread Dave Brosius
Greetings, org.apache.jackrabbit.webdav.security.report.AclPrincipalReport.java does the following This logic seems wrong to me *if (href == null && respMap.containsKey(href)) {* i'd expect it should be || correct? @Override public void init(DavResource resource, ReportInfo info) thr

Re: Seemingly incorrect short-circuiting logic

2011-10-16 Thread sebb
Wrong mailing list? This is Jakarta, not Jackrabbit ... On 17 October 2011 00:44, Dave Brosius wrote: > Greetings, > > org.apache.jackrabbit.webdav.security.report.AclPrincipalReport.java > > does the following > > This logic seems wrong to me > > *if (href == null && respMap.containsKey(href))

Re: Seemingly incorrect short-circuiting logic

2011-10-16 Thread Dave Brosius
Oops, my apologies :) On 10/16/2011 08:06 PM, sebb wrote: Wrong mailing list? This is Jakarta, not Jackrabbit ... On 17 October 2011 00:44, Dave Brosius wrote: Greetings, org.apache.jackrabbit.webdav.security.report.AclPrincipalReport.java does the following This logic seems wrong to me