[jira] Created: (QPID-3024) Standardise Qpid concurrency utils by using classes from java.util.concurrent

2011-01-31 Thread Andrew Kennedy (JIRA)
Standardise Qpid concurrency utils by using classes from java.util.concurrent - Key: QPID-3024 URL: https://issues.apache.org/jira/browse/QPID-3024 Project: Qpid

[jira] Assigned: (QPID-3024) Standardise Qpid concurrency utils by using classes from java.util.concurrent

2011-01-31 Thread Andrew Kennedy (JIRA)
[ https://issues.apache.org/jira/browse/QPID-3024?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrew Kennedy reassigned QPID-3024: Assignee: Andrew Kennedy Standardise Qpid concurrency utils by using classes from

[jira] Created: (QPID-3025) Upgrade Java test framework to JUnit 4

2011-01-31 Thread Andrew Kennedy (JIRA)
Upgrade Java test framework to JUnit 4 -- Key: QPID-3025 URL: https://issues.apache.org/jira/browse/QPID-3025 Project: Qpid Issue Type: Improvement Reporter: Andrew Kennedy

[jira] Created: (QPID-3026) The ApplicationRegistry object should be a singleton

2011-01-31 Thread Andrew Kennedy (JIRA)
The ApplicationRegistry object should be a singleton Key: QPID-3026 URL: https://issues.apache.org/jira/browse/QPID-3026 Project: Qpid Issue Type: Improvement Components: Java

RE: Zap trailing whitespace?

2011-01-31 Thread Steve Huston
Hi Jonathan, I'd like to zap trailing whitespace from the source files in our SVN repo. Does anyone object to me doing this? Do we agree that source files should not have trailing whitespace? I agree - I think zapping whitespace is a good idea and would welcome your efforts to do so.

Re: Zap trailing whitespace?

2011-01-31 Thread Andrew Stitcher
On Mon, 2011-01-31 at 15:36 -0500, Jonathan Robie wrote: I'd like to zap trailing whitespace from the source files in our SVN repo. Does anyone object to me doing this? Yes I object. This will cause spurious diffs to pretty much every C++ file, and may well cause failed merges. The diffs will

Re: Zap trailing whitespace?

2011-01-31 Thread Jonathan Robie
On 01/31/2011 04:03 PM, Andrew Stitcher wrote: The way I'd prefer to deal with this is to add a hook to the svn checkin process (and while we're doing that ban tab characters too; ban them because there is no way to know how many spaces they represent universally) which would strip them out of

RE: Zap trailing whitespace?

2011-01-31 Thread Steve Huston
For the record, I like Andrew's approach better... Hook the svn commit to check for spaces/tabs. -Steve -Original Message- From: Andrew Stitcher [mailto:astitc...@redhat.com] Sent: Monday, January 31, 2011 4:03 PM To: dev@qpid.apache.org Subject: Re: Zap trailing whitespace?

Re: Zap trailing whitespace?

2011-01-31 Thread Andrew Stitcher
On Mon, 2011-01-31 at 16:38 -0500, Jonathan Robie wrote: .. If we do the svn hook, but don't strip trailing whitespace in one checkin, we get a lot of spurious diffs over a longer period of time, and they aren't related to the purpose of a given check in. I think you may be

[jira] Commented: (QPID-3009) Perl binding to Qpid messaging

2011-01-31 Thread Gordon Sim (JIRA)
[ https://issues.apache.org/jira/browse/QPID-3009?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12988926#comment-12988926 ] Gordon Sim commented on QPID-3009: -- Thanks for that suggestion Jose. I've committed a

Re: Zap trailing whitespace?

2011-01-31 Thread Jonathan Robie
On 01/31/2011 04:43 PM, Steve Huston wrote: For the record, I like Andrew's approach better... Hook the svn commit to check for spaces/tabs. OK, let me try for consensus around this: Does anyone object to adding an svn hook that (1) strips trailing whitespace, and (2) bans commits that

Re: Zap trailing whitespace?

2011-01-31 Thread Jonathan Robie
On 01/31/2011 04:53 PM, Andrew Stitcher wrote: [I'm not against people incrementally tidying up whitespace pollution in files that they touch beyond whatever changes they make, but I am objecting to doing it in a massive single checkin] OK, unless someone gripes, I think I'll keep the emacs

Re: Zap trailing whitespace?

2011-01-31 Thread Justin Ross
On Mon, 31 Jan 2011, Andrew Stitcher wrote: [I'm not against people incrementally tidying up whitespace pollution in files that they touch beyond whatever changes they make, but I am objecting to doing it in a massive single checkin] I'm not terribly fond of larding cleanups in with other

RE: Zap trailing whitespace?

2011-01-31 Thread Steve Huston
-Original Message- From: Jonathan Robie [mailto:jonathan.ro...@redhat.com] Sent: Monday, January 31, 2011 5:32 PM To: dev@qpid.apache.org Subject: Re: Zap trailing whitespace? On 01/31/2011 04:53 PM, Andrew Stitcher wrote: [I'm not against people incrementally tidying up

RE: Zap trailing whitespace?

2011-01-31 Thread Steve Huston
OK, let me try for consensus around this: Does anyone object to adding an svn hook that (1) strips trailing whitespace, and (2) bans commits that contain tabs? I think I'd rather see the hook refuse to do the commit if there are trailing whitespace or tabs. On the off chance that svn gets

Re: Zap trailing whitespace?

2011-01-31 Thread Andrew Stitcher
On Mon, 2011-01-31 at 17:31 -0500, Jonathan Robie wrote: On 01/31/2011 04:43 PM, Steve Huston wrote: For the record, I like Andrew's approach better... Hook the svn commit to check for spaces/tabs. OK, let me try for consensus around this: Does anyone object to adding an svn hook that

[jira] Commented: (QPID-3009) Perl binding to Qpid messaging

2011-01-31 Thread Jose Pedro Oliveira (JIRA)
[ https://issues.apache.org/jira/browse/QPID-3009?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12988981#comment-12988981 ] Jose Pedro Oliveira commented on QPID-3009: --- Gordon, Thanks for the update. The

Re: Zap trailing whitespace?

2011-01-31 Thread Chuck Rolke
How many files are we talking about here? Yes, spurious trailing-white-space diffs are bad but if there's only 20-30 files then I could live with that. My bet is that there are way more files with embedded tabs. I'll vote yes for a hook that flags/complains about a commit with tabs or