[ANNOUNCE] vdr-sudoku-0.2.0

Message ID 200803221424.13987.ville.skytta@iki.fi
State New
Headers

Commit Message

Ville Skyttä March 22, 2008, 12:24 p.m. UTC
  On Saturday 22 March 2008, Thomas Günther wrote:
> Hi!
>
> There is a new version of the Sudoku plug-in.
>
> Download: http://toms-cafe.de/vdr/sudoku/vdr-sudoku-0.2.0.tgz

Here's a bunch of patches:

- finnish: Add missing Finnish translations.

- no-pattern-default: Turn of "show numbers as pattern" by default; I find the 
feature quite unexpected and it causes a mess (bleeding colors) with my DXR3 
OSD.

- no-exit: Remove the "Exit" choice from the commands menu; exiting the plugin 
using this menu entry exits the plugin fine but next time any VDR OSD should 
appear after doing this, VDR crashes (VDR 1.4.7, Fedora Core 8).  This is 
obviously just a stopgap patch, it'd be better if this could be fixed 
properly.
  

Comments

Ville Skyttä March 23, 2008, 9:40 a.m. UTC | #1
On Saturday 22 March 2008, Thomas Günther wrote:
> Ville Skyttä wrote:
>
> > - no-exit: Remove the "Exit" choice from the commands menu; exiting the
> > plugin using this menu entry exits the plugin fine but next time any VDR
> > OSD should appear after doing this, VDR crashes (VDR 1.4.7, Fedora Core
> > 8).  This is obviously just a stopgap patch, it'd be better if this could
> > be fixed properly.
>
> Sorry, this was a last minute change, without testing the exit
> function. :-( Please try the attached patch. It solves the problem for
> me.

Seems to work for me too, thanks.
  

Patch

Don't show possible numbers as a pattern by default, it's an unexpected
feature and doesn't work properly with all OSD's.

diff -up sudoku-0.2.0/setup.cpp~ sudoku-0.2.0/setup.cpp
--- sudoku-0.2.0/setup.cpp~	2008-03-22 04:00:00.000000000 +0200
+++ sudoku-0.2.0/setup.cpp	2008-03-22 13:43:53.000000000 +0200
@@ -39,7 +39,7 @@  SetupData::SetupData()
   symmetric = 1;
   mark_errors = 1;
   mark_ambiguous = 1;
-  show_possibles_pattern = 1;
+  show_possibles_pattern = 0;
   show_possibles_digits = 1;
   clear_marks = 0;
   transparency = 50;