rpms/eclipse-valgrind/devel eclipse-valgrind-locale-percentage-strings.patch, NONE, 1.1 eclipse-valgrind.spec, 1.3, 1.4

[Elliott Baron]

Author: ebaron

Update of /cvs/pkgs/rpms/eclipse-valgrind/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv16841

Modified Files:
	eclipse-valgrind.spec 
Added Files:
	eclipse-valgrind-locale-percentage-strings.patch 
Log Message:
* Fri Mar 13 2009 Elliott Baron <ebaron at redhat.com> 0.1.0-4
- Fixed Massif parser crashing on other locales.

eclipse-valgrind-locale-percentage-strings.patch:

--- NEW FILE eclipse-valgrind-locale-percentage-strings.patch ---
### Eclipse Workspace Patch 1.0
#P org.eclipse.linuxtools.valgrind.massif
Index: src/org/eclipse/linuxtools/valgrind/massif/MassifParser.java
===================================================================
--- src/org/eclipse/linuxtools/valgrind/massif/MassifParser.java	(revisión: 21664)
+++ src/org/eclipse/linuxtools/valgrind/massif/MassifParser.java	(copia de trabajo)
@@ -140,7 +140,7 @@
 		else {
 			percentage = numBytes.doubleValue() / snapshot.getTotal() * 100;
 		}
-		nodeText.append(Double.valueOf(new DecimalFormat("0.##").format(percentage)) + "%"); //$NON-NLS-1$ //$NON-NLS-2$
+		nodeText.append(new DecimalFormat("0.##").format(percentage) + "%"); //$NON-NLS-1$ //$NON-NLS-2$
 		nodeText.append(" ("); //$NON-NLS-1$
 		nodeText.append(new DecimalFormat("#,##0").format(numBytes.longValue()) + "B"); //$NON-NLS-1$ //$NON-NLS-2$
 		nodeText.append(")"); //$NON-NLS-1$
Index: ChangeLog
===================================================================
--- ChangeLog	(revisión: 21664)
+++ ChangeLog	(copia de trabajo)
@@ -1,3 +1,7 @@
+2009-03-13  Elliott Baron  <ebaron at redhat.com>
+
+	* MassifParser.java (parseTree): Fixed NFE on localized percentage strings.
+
 2009-01-20  Elliott Baron  <ebaron at redhat.com>
 
 	* MassifLaunchDelegate: Changed to handle multiple pids.


Index: eclipse-valgrind.spec
===================================================================
RCS file: /cvs/pkgs/rpms/eclipse-valgrind/devel/eclipse-valgrind.spec,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- eclipse-valgrind.spec	6 Mar 2009 16:35:01 -0000	1.3
+++ eclipse-valgrind.spec	13 Mar 2009 16:16:37 -0000	1.4
@@ -4,7 +4,7 @@
 
 Name:           eclipse-valgrind
 Version:        0.1.0
-Release:        3%{?dist}
+Release:        4%{?dist}
 Summary:        Valgrind Tools Integration for Eclipse
 
 Group:          Development/Tools
@@ -12,6 +12,7 @@
 URL:            http://www.eclipse.org/linuxtools/projectPages/valgrind
 Source0:        %{name}-fetched-src-%{src_repo_tag}.tar.bz2
 Source1:        %{name}-fetch-src.sh
+Patch0:         %{name}-locale-percentage-strings.patch
 BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
 #No CDT on ppc64
@@ -34,6 +35,9 @@
 
 %prep
 %setup -q -n %{name}-fetched-src-%{src_repo_tag}
+pushd org.eclipse.linuxtools.valgrind.massif
+%patch0
+popd
 
 %build
 %{eclipse_base}/buildscripts/pdebuild \
@@ -57,6 +61,9 @@
 %doc org.eclipse.linuxtools.valgrind-feature/epl-v10.html
 
 %changelog
+* Fri Mar 13 2009 Elliott Baron <ebaron at redhat.com> 0.1.0-4
+- Fixed Massif parser crashing on other locales.
+
 * Fri Mar 6 2009 Elliott Baron <ebaron at redhat.com> 0.1.0-3
 - Changing to arch dependent for CDT dependency.
 - Setting minimum Valgrind requirement to 3.3.0.