<div dir="ltr">Hi Dan,<div>Thanks again,</div><div>I sent a new patch with a validation routine and an exception upon error</div><div><br></div><div>Ran</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 12, 2022 at 5:37 PM Dan Smith via chirp_devel <<a href="mailto:chirp_devel@intrepid.danplanet.com">chirp_devel@intrepid.danplanet.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">> I will modify the fix to have a scan / validate routine as you suggested.<br>
> Another option would be to have a dynamic value list for the setting (one that is update at every memory access, (e.g. similar to my cached priority list)?<br>
> As far as I can tell the value lists are static (?) <br>
<br>
I'm not sure what you mean exactly. The drivers should work the same way regardless of how or what order they're called. Meaning something like chirpc that won't fetch or set every memory before letting you tweak one should have the same behavior as the UI which does. The list depends on the current state of the memories, which is why it's not a static list at the top of the file, but built from the memories on the fly, right?<br>
<br>
You could build the cached list the first time you need it (if it's not built yet) and keep it coherent with each subsequent get/set call if you want. But, I suspect it's such a tiny amount of data that it's not worth doing that versus just scanning each time, but up to you. I just think it's not reasonable to say "meh, the UI will load all the memories before a set_memory() is ever called so I will have built my list by then" :)<br>
<br>
--Dan<br>
_______________________________________________<br>
chirp_devel mailing list<br>
<a href="mailto:chirp_devel@intrepid.danplanet.com" target="_blank">chirp_devel@intrepid.danplanet.com</a><br>
<a href="http://intrepid.danplanet.com/mailman/listinfo/chirp_devel" rel="noreferrer" target="_blank">http://intrepid.danplanet.com/mailman/listinfo/chirp_devel</a><br>
Developer docs: <a href="http://chirp.danplanet.com/projects/chirp/wiki/Developers" rel="noreferrer" target="_blank">http://chirp.danplanet.com/projects/chirp/wiki/Developers</a><br>
</blockquote></div>