<div dir="ltr"><br><div class="gmail_quote">On Mon, Aug 2, 2010 at 2:13 PM, Ilkka Ollakka <span dir="ltr"><<a href="mailto:ileoo@videolan.org">ileoo@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On Mon, Aug 02, 2010 at 01:21:40PM +0300, David Kaplan wrote:<br>
> DVB satellite scanning using transponder lists from the config directory<br>
<br>
</div>Does those satelites really change that often that they need<br>
config-file, or is there so many of those?<br><div class="im"></div></blockquote><div><br>There are a LOT of them.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">
>  /* Satellite */<br>
> +#define SATELLITE_TEXT N_("Satellite scanning config")<br>
> +#define SATELLITE_LONGTEXT N_("filename of config file in share/dvb/dvb-s")<br>
<br>
</div>Does this mean it only needs filename, or full path with filename?<br></blockquote><div><br>Only filename - will make it clearer<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div class="im"><br>
> -#define DVB_SCAN_MAX_SIGNAL_TIME (300*1000)<br>
> +#define DVB_SCAN_MAX_SIGNAL_TIME (1000*1000)<br>
>  #define DVB_SCAN_MAX_LOCK_TIME (5000*1000)<br>
> -#define DVB_SCAN_MAX_PROBE_TIME (30000*1000)<br>
> +#define DVB_SCAN_MAX_PROBE_TIME (45000*1000)<br>
<br>
</div>Doesn't this slowdown all the other scannings ?<br><div class="im"></div></blockquote><div><br>Yes it does but I'm not sure that's a bad thing. My tests showed a LOT of unnecessary timeouts with lower signal times.<br>
I think that this is a good compromise.<br><br>Also as part of my ongoing dvb improvements, I hope to make persisting of the canned lists more logical so scanning isn't performed very often.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">
> -static int ScanDvbTNextExhaustive( scan_t *p_scan, scan_configuration_t *p_cfg, double *pf_pos )<br>
> +static int ScanDvbNextExhaustive( scan_t *p_scan, scan_configuration_t *p_cfg, double *pf_pos )<br>
<br>
</div>I think it would be clearer to make separated patch that does<br>
non dvb-s scanning related changes.<font color="#888888"><br></font></blockquote><div><br>I can do this - problem is that I've done it all together already and now I'd have to manually go and find each place and split it up.<br>
The changes were really minor so is this strictly necessary?<br clear="all"></div></div><br>-- <br><div dir="ltr"><font size="1"><a style="font-family: courier new,monospace;" href="http://blog.2of1.org" target="_blank">http://blog.2of1.org</a></font><br>
</div><br>
</div>