Fix comments on functions to make them clearer

Author: MatteoPologruto

arduino/httpclient/httpclient.go

@@ -33,6 +33,7 @@ var tr = i18n.Tr
 
 // DownloadFile downloads a file from a URL into the specified path. An optional config and options may be passed (or nil to use the defaults).
 // A DownloadProgressCB callback function must be passed to monitor download progress.
+// If a not empty queryParameter is passed, it is appended to the URL for analysis purposes.
 func DownloadFile(path *paths.Path, URL string, queryParameter string, label string, downloadCB rpc.DownloadProgressCB, config *downloader.Config, options ...downloader.DownloadOptions) (returnedError error) {
 	if queryParameter != "" {
 		URL = URL + "?query=" + queryParameter

arduino/resources/download.go

@@ -27,6 +27,7 @@ import (
 
 // Download performs a download loop using the provided downloader.Config.
 // Messages are passed back to the DownloadProgressCB using label as text for the File field.
+// queryParameter is passed for analysis purposes.
 func (r *DownloadResource) Download(downloadDir *paths.Path, config *downloader.Config, label string, downloadCB rpc.DownloadProgressCB, queryParameter string) error {
 	path, err := r.ArchivePath(downloadDir)
 	if err != nil {

commands/lib/download.go

@@ -30,7 +30,9 @@ import (
 
 var tr = i18n.Tr
 
-// LibraryDownload FIXMEDOC
+// LibraryDownload executes the download of the library.
+// A DownloadProgressCB callback function must be passed to monitor download progress.
+// queryParameter is passed for analysis purposes.
 func LibraryDownload(ctx context.Context, req *rpc.LibraryDownloadRequest, downloadCB rpc.DownloadProgressCB, queryParameter string) (*rpc.LibraryDownloadResponse, error) {
 	logrus.Info("Executing `arduino-cli lib download`")
 

commands/lib/install.go

@@ -30,7 +30,8 @@ import (
 	"github.com/sirupsen/logrus"
 )
 
-// LibraryInstall FIXMEDOC
+// LibraryInstall resolves the library dependencies, then downloads and installs the libraries into the install location.
+// queryParameter is passed for analysis purposes and forwarded to the called functions. It is set to "depends" when a dependency is installed
 func LibraryInstall(ctx context.Context, req *rpc.LibraryInstallRequest, downloadCB rpc.DownloadProgressCB, taskCB rpc.TaskProgressCB, queryParameter string) error {
 	lm := commands.GetLibraryManager(req)
 	if lm == nil {
@@ -96,6 +97,7 @@ func LibraryInstall(ctx context.Context, req *rpc.LibraryInstallRequest, downloa
 	}
 
 	for libRelease, installTask := range libReleasesToInstall {
+		// Checks if libRelease is the requested library and not a dependency
 		if libRelease.GetName() == req.Name {
 			if err := downloadLibrary(lm, libRelease, downloadCB, taskCB, queryParameter); err != nil {
 				return err