KeepAlive() change request in the api

rfoust

In radio.cs, the KeepAlive() function has an if block to check if:

if (!API.ProgramName.Contains("Maestro"))

Any chance that can be changed to check for the existence of ProgramName to prevent a null reference exception?  Something like this:

if ((API.ProgramName != null && API.ProgramName != "") && !API.ProgramName.Contains("Maestro"))

Thanks!

Mark_W3II

I am guessing they do not check for this because you should set this value before calling Init.
Here is what I do for FRStack

API.ProgramName = "FRStack";
API.Init();

rfoust

Yeah I've modified my code to do that now, just took a sec to figure out why it was blowing up. Either way, not a huge deal.

Now I gotta figure out why keepalive isn't actually keeping it alive. Haven't played with the API in a few years...It has changed quite a bit. :-)

Eric-KE5DTO

Note that KeepAlive doesn't run in Debug mode.  Only in Release.

Eric-KE5DTO

Indeed, a ProgramName should be set before calling API.Init.  However, it could be checked and something more sensible shown to help the developer figure this out.  I've entered issue #5528 to address this.  Thanks for the report.