Eclipse Community Forums
Forum Search:

Search      Help    Register    Login    Home
Home » Archived » Memory Analyzer (MAT) » Minor CSVOutputter patch - 1 attachment
Minor CSVOutputter patch - 1 attachment [message #7185] Wed, 19 November 2008 20:19 Go to next message
Eclipse UserFriend
Originally posted by: cgrinds.us.ibm.com

I noticed that Export to CSV uses the semicolon as the field separator. I
understand this is the standard in parts (all?) of Europe, but isn't the
standard in the US. This patch asks the Locale and tries to use the most
approriate format.

CSV is an ill defined spec.


--
Chris Grindstaff
cgrinds@us.ibm.com

Re: Minor CSVOutputter patch - 1 attachment [message #7573 is a reply to message #7185] Thu, 20 November 2008 12:29 Go to previous messageGo to next message
Andreas Buchen is currently offline Andreas BuchenFriend
Messages: 113
Registered: July 2009
Senior Member
> I noticed that Export to CSV uses the semicolon as the field separator. I
> understand this is the standard in parts (all?) of Europe, but isn't the
> standard in the US. This patch asks the Locale and tries to use the most
> approriate format.

Hi Chris, for one reason or another, I cannot read the attached patch...
:-(

But your suggestion makes absolutely sense. I know changed the code to

private static final char SEPARATOR = new
DecimalFormatSymbols().getDecimalSeparator() == ',' ? ';' : ':';

I read that countries using the comma as decimal point are the ones that
use the semicolon as separator char.

Is this in line with the patch you suggest?

Andreas.
Re: Minor CSVOutputter patch - 1 attachment [message #7594 is a reply to message #7573] Thu, 20 November 2008 12:30 Go to previous messageGo to next message
Andreas Buchen is currently offline Andreas BuchenFriend
Messages: 113
Registered: July 2009
Senior Member
And, yes, it must read:

private static final char SEPARATOR = new
DecimalFormatSymbols().getDecimalSeparator() == ',' ? ';' : ',';
Re: Minor CSVOutputter patch - 1 attachment [message #7615 is a reply to message #7594] Thu, 20 November 2008 13:16 Go to previous message
Eclipse UserFriend
Originally posted by: cgrinds.us.ibm.com

andreas.buchen@sap.com (Andreas Buchen) wrote in
news:b806027f876b07d023ec6ba2db31aabf$1@www.eclipse.org:

> And, yes, it must read:
>
> private static final char SEPARATOR = new
> DecimalFormatSymbols().getDecimalSeparator() == ',' ? ';' : ',';
>

Hello Andreas,
Yes that's exactly my patch.
Thanks for applying,
Chris
--
Chris Grindstaff
cgrinds@us.ibm.com
Previous Topic:Save Value to File
Next Topic:New inspectors
Goto Forum:
  


Current Time: Wed Nov 26 21:43:54 GMT 2014

Powered by FUDForum. Page generated in 0.02003 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software