Subject: aldo: Setup options expect extra Enter before starting
Package: aldo
Version: 0.7.6-1
Severity: minor
Tags: patch

When choosing option 5 (setup), then any option ranging from 1 to 4, aldo
doesn't show anything, and waits for the user to press Enter before going on.
One would expect to immediatly get the first question like "Pause before keying
(seconds) [2]:", or at least some thing like "press enter to start".

This is simply due to extra cin.ignore() lines (see patch)



-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)

Kernel: Linux 2.6.32-5-686 (SMP w/2 CPU cores)
Locale: LANG=en_GB.utf8, LC_CTYPE=en_GB.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages aldo depends on:
ii  libao4                        1.0.0-5    Cross Platform Audio Output Librar
ii  libc6                         2.11.2-7   Embedded GNU C Library: Shared lib
ii  libgcc1                       1:4.4.5-8  GCC support library
ii  libstdc++6                    4.4.5-8    The GNU Standard C++ Library v3

aldo recommends no packages.

aldo suggests no packages.

-- no debconf information
diff -Naur aldo-0.7.6.keep//src/setup.cc aldo-0.7.6/src/setup.cc
--- aldo-0.7.6.keep//src/setup.cc	2010-01-16 17:56:30.000000000 +0100
+++ aldo-0.7.6/src/setup.cc	2010-12-27 13:01:52.000000000 +0100
@@ -115,8 +115,6 @@
     Datafile cfg = get_configuration();
     sec_it this_sec = cfg.section("Keyer");
     
-    cin.ignore();
-
     update_option(this_sec->option("BeginPause"), insert_beginpause);
     update_option(this_sec->option("CharPause"), insert_charpause);
     update_option(this_sec->option("StringPause"), insert_strpause);
@@ -132,8 +130,6 @@
     Datafile cfg = get_configuration();
     sec_it this_sec = cfg.section("Blocks");
 
-    cin.ignore();
-
     update_option(this_sec->option("StringsNumber"), insert_strnum);
 
     update_option(this_sec->option("StringLength"), insert_strlen);
@@ -182,8 +178,6 @@
     Datafile cfg = get_configuration();
     sec_it this_sec = cfg.section("Koch");
 
-    cin.ignore();
-
     update_option(this_sec->option("Chars"), insert_chars);
 
     update_option(this_sec->option("StringsNumber"), insert_strnum);
@@ -202,8 +196,6 @@
     Datafile cfg = get_configuration();
     sec_it this_sec = cfg.section("Qrz");
 	
-    cin.ignore();
-
     update_option(this_sec->option("CallFormat"), insert_callformat);
     update_option(this_sec->option("StringsNumber"), insert_strnum);
 }

Reply via email to