Klaus Schmidinger wrote:
...
> it would appear to me that this won't work if no SourceCaps are set.
> If you really want this to go into VDR then you need to make sure
> a system that does not have SourceCaps set (which IMHO will be the
> majority) will still work.
I fully agree that the default must be "source is provided".
I suppose a global flag "SourceCapsSet" which is initialized
to false and which is set true true by ParseSourceCaps should
do the trick?
After all, this is Christian's patch, so I do not want to take
it over unless Christian wants me to.
Christian, are you reading this?
Would you like to fix your patch so Klaus can accept it?
>
> BTW: what's with this:
>
> diff -ru /hetis/home/cko/VDR/sources.c VDR/sources.c
> --- /hetis/home/cko/VDR/sources.c 2004-12-26 12:58:52.000000000 +0100
> +++ VDR/sources.c 2005-04-05 22:12:55.421326944 +0200
> @@ -68,7 +68,7 @@
> int pos = 0;
> bool dot = false;
> bool neg = false;
> - while (*++s) {
> + while (*++s && !isblank(*s)) {
> switch (toupper(*s)) {
> case '0' ... '9': pos *= 10;
> pos += *s - '0';
>
> Obviously this has nothing to do with SourceCaps - but what's it for then?
I do not know.
My guess is that Christian needed this in a previous version of
cSetup::ParseSourceCaps? After all, cSetup::ParseSourceCaps is
calling cSource::FromString.
I believe this hunk can safely be omitted.
Carsten.
