Suppress mistaken LGTM warnings

RELEASE_NOTES.md

@@ -16,7 +16,7 @@ This file contains a high-level description of this package's evolution. Release
 
 #### Changes
 
-* [Bug Fix] Get remotetest server working for DAP4 and re-enable its use in the netcdf-c library for testing. See [Github #????](https://github.com/Unidata/netcdf-c/pull/????).
+* [Bug Fix] Get remotetest server working for DAP4 and re-enable its use in the netcdf-c library for testing. See [Github #2558](https://github.com/Unidata/netcdf-c/pull/2558).
 * [Bug Fix] Fix a race condition when testing missing filters. See [Github #2557](https://github.com/Unidata/netcdf-c/pull/2557). 
 * [Bug Fix] Make major changes to libdap4 and dap4_test to update the non-remote DAP4 tests. See [Github #2555](https://github.com/Unidata/netcdf-c/pull/2555).
 * [Bug Fix] Fix some race conditions due to use of a common file in multiple shell scripts . See [Github #2552](https://github.com/Unidata/netcdf-c/pull/2552).

include/nctestserver.h

@@ -102,7 +102,7 @@ nc_findtestserver(const char* path, const char* serverlist)
 	if(reportsearch)
 	    fprintf(stderr,"nc_findtestserver: candidate=%s/%s: found=",*svc,path);
 	/* Try http: first */
-        snprintf(url,MAXSERVERURL,"http://%s/%s",*svc,path);
+        snprintf(url,MAXSERVERURL,"http://%s/%s",*svc,path); /*lgtm [cpp/non-https-url]*/
 	if(ping(url) == NC_NOERR) {
 	    if(reportsearch) fprintf(stderr,"yes\n");
 	    match = strdup(url);

ncdap_test/pingurl.c

@@ -41,7 +41,7 @@ main(int argc, char** argv)
 	usage();
 
     /* Try http: first */
-    snprintf(url,MAXSERVERURL,"http://%s",argv[0]);
+    snprintf(url,MAXSERVERURL,"http://%s",argv[0]); /* lgtm [cpp/non-https-url] */
     if(timedping(url,PINGTIME) == NC_NOERR) 
 	found = 1;
     else {