Re: [PATCH] Diseqc Positioner support for VDR 0.66

[Klaus Schmidinger <Klaus.Schmidinger@cadsoft.de>]

Niels de Carpentier wrote:
> 
> Hi,
> 
> I ported my old patch for diseqc positioner support to the latest VDR version
> (0.66). This version of the patch uses the grouping separators to store the
> satellite number. The group separator now has the following format:
> ":group name:satelite number". All the channels below a group separator have
> satellite number of the group separator.

Hello Niels,

I just took a quick look at your patch.
By adding the sat number to the group delimiter you are restricting
the usefulness of the group delimiter quite a bit. The original
idea was to allow grouping _any_ channels (not necessarily on the
same satellite!) together. With your modification grouped channels
_must_ be on the same satellite.

I'll adopt your sat positioner code once I have more time, but I
guess I'll introduce an additional "satNr" field in the channels.conf
lines.

Greetings
Klaus
-- 
_______________________________________________________________

Klaus Schmidinger                       Phone: +49-8635-6989-10
CadSoft Computer GmbH                   Fax:   +49-8635-6989-40
Hofmark 2                               Email:   kls@cadsoft.de
D-84568 Pleiskirchen, Germany           URL:     www.cadsoft.de
_______________________________________________________________