X-Git-Url: https://gitweb.fperrin.net/?a=blobdiff_plain;f=tim%2Fprune%2Fsave%2FExifSaver.java;h=42f9e4f80cf4c3dee51d6478eaa792ca604af6fa;hb=63f178fd6c6b30b99a01f2a2d700963ea2dfef8b;hp=1e597201d5f2842197531159ec3452f542fb428c;hpb=5625a1abadb5f2ca5f017fe7dbda1d5141cb637b;p=GpsPrune.git

diff --git a/tim/prune/save/ExifSaver.java b/tim/prune/save/ExifSaver.java
index 1e59720..42f9e4f 100644
--- a/tim/prune/save/ExifSaver.java
+++ b/tim/prune/save/ExifSaver.java
@@ -35,9 +35,11 @@ public class ExifSaver implements Runnable
 {
 	private Frame _parentFrame = null;
 	private JDialog _dialog = null;
+	private JButton _okButton = null;
 	private JCheckBox _overwriteCheckbox = null;
 	private JProgressBar _progressBar = null;
 	private PhotoTableModel _photoTableModel = null;
+	private boolean _saveCancelled = false;
 
 
 	// To preserve timestamps of file use parameter -P
@@ -72,6 +74,7 @@ public class ExifSaver implements Runnable
 	 * Save exif information to all photos in the list
 	 * whose coordinate information has changed since loading
 	 * @param inPhotoList list of photos to save
+	 * @return true if saved
 	 */
 	public boolean saveExifInformation(PhotoList inPhotoList)
 	{
@@ -166,19 +169,22 @@ public class ExifSaver implements Runnable
 		// Lower panel with ok and cancel buttons
 		JPanel buttonPanel = new JPanel();
 		buttonPanel.setLayout(new FlowLayout(FlowLayout.RIGHT));
-		JButton okButton = new JButton(I18nManager.getText("button.ok"));
-		okButton.addActionListener(new ActionListener() {
+		_okButton = new JButton(I18nManager.getText("button.ok"));
+		_okButton.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e)
 			{
+				// disable ok button
+				_okButton.setEnabled(false);
 				// start new thread to do save
 				new Thread(ExifSaver.this).start();
 			}
 		});
-		buttonPanel.add(okButton);
+		buttonPanel.add(_okButton);
 		JButton cancelButton = new JButton(I18nManager.getText("button.cancel"));
 		cancelButton.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e)
 			{
+				_saveCancelled = true;
 				_dialog.dispose();
 			}
 		});
@@ -208,6 +214,7 @@ public class ExifSaver implements Runnable
 	 */
 	public void run()
 	{
+		_saveCancelled = false;
 		PhotoTableEntry entry = null;
 		Photo photo = null;
 		int numPhotos = _photoTableModel.getRowCount();
@@ -220,7 +227,7 @@ public class ExifSaver implements Runnable
 		for (int i=0; i<numPhotos; i++)
 		{
 			entry = _photoTableModel.getPhotoTableEntry(i);
-			if (entry != null && entry.getSaveFlag())
+			if (entry != null && entry.getSaveFlag() && !_saveCancelled)
 			{
 				// Only look at photos which are selected and whose status has changed since load
 				photo = entry.getPhoto();
@@ -297,7 +304,12 @@ public class ExifSaver implements Runnable
 		// Execute exif command
 		try
 		{
-			Runtime.getRuntime().exec(command);
+			Process process = Runtime.getRuntime().exec(command);
+			// Wait for process to finish so not too many run in parallel
+			try {
+				process.waitFor();
+			}
+			catch (InterruptedException ie) {}
 		}
 		catch (Exception e)
 		{