From 6001254160f00e6766681554f8d82260a9c0a62a Mon Sep 17 00:00:00 2001
From: Dick Hollenbeck <dick@softplc.com>
Date: Thu, 18 Apr 2013 12:03:25 -0500
Subject: [PATCH] fix Lorenzo's bug introduced in rev. 4082

---
 common/dialogs/dialog_page_settings.cpp | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/common/dialogs/dialog_page_settings.cpp b/common/dialogs/dialog_page_settings.cpp
index 20aca80eb9..18fe9c7366 100644
--- a/common/dialogs/dialog_page_settings.cpp
+++ b/common/dialogs/dialog_page_settings.cpp
@@ -63,11 +63,10 @@ const wxString pageFmts[] =
     _("C 17x22in"),
     _("D 22x34in"),
     _("E 34x44in"),
-    _("US Letter 8.5x11in"),
-    _("US Legal 8.5x14in"),
-    _("US Ledger 11x17in"),
+    _("USLetter 8.5x11in"),     // USLetter without space is correct
+    _("USLegal 8.5x14in"),      // USLegal without space is correct
+    _("USLedger 11x17in"),      // USLedger without space is correct
     _("User (Custom)"),
-    wxT("")                 // end of list
 };
 
 void EDA_DRAW_FRAME::Process_PageSettings( wxCommandEvent& event )
@@ -115,11 +114,8 @@ void DIALOG_PAGES_SETTINGS::initDialog()
     // The first shows translated strings, the second contains not translated strings
     m_paperSizeComboBox->Clear();
 
-    for( unsigned ii = 0; ; ii++ )
+    for( unsigned ii = 0; ii<DIM(pageFmts); ii++ )
     {
-        if( pageFmts[ii].IsEmpty() )
-            break;
-
         m_pageFmt.Add( pageFmts[ii] );
         m_paperSizeComboBox->Append( wxGetTranslation( pageFmts[ii] ) );
     }