Clean comments in iconTools.js

Lucas Rojas · Lucas Rojas · commit e16428609298 · 2019-02-15T17:11:06.000-03:00
diff --git a/lib/iconTools.js b/lib/iconTools.js
@@ -17,7 +17,6 @@ function getIcon(iconUrl, iconFilePath, callback) {
 	return fileTools.mkdirp(iconFolder).then(function () {
     return download(iconUrl, iconFilePath).then(function (response) {
       if (response.statusCode === 200 && !response.contentType.match(/image/)) {
-        // If not an image, return an error
         return Q.reject(new Error('Downloaded file  \'' + iconUrl + '\' does not appear to be an image. The content type of the response was \'' + response.contentType + '\'.'));
       }
       
@@ -29,7 +28,6 @@ function getIcon(iconUrl, iconFilePath, callback) {
 
 function copyDefaultIcon(manifest, platformId, iconSize, source, targetPath, callback) {
 	
-	// platform already contains an icon with this size - skip
 	if (manifest.icons && manifest.icons[iconSize]) {
 		return Q.resolve().nodeify(callback);
 	}

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,6 @@ function getIcon(iconUrl, iconFilePath, callback) {`
`17`	`17`	`return fileTools.mkdirp(iconFolder).then(function () {`
`18`	`18`	`return download(iconUrl, iconFilePath).then(function (response) {`
`19`	`19`	`if (response.statusCode === 200 && !response.contentType.match(/image/)) {`
`20`		`- // If not an image, return an error`
`21`	`20`	`return Q.reject(new Error('Downloaded file \'' + iconUrl + '\' does not appear to be an image. The content type of the response was \'' + response.contentType + '\'.'));`
`22`	`21`	`}`
`23`	`22`
`@@ -29,7 +28,6 @@ function getIcon(iconUrl, iconFilePath, callback) {`
`29`	`28`
`30`	`29`	`function copyDefaultIcon(manifest, platformId, iconSize, source, targetPath, callback) {`
`31`	`30`
`32`		`- // platform already contains an icon with this size - skip`
`33`	`31`	`if (manifest.icons && manifest.icons[iconSize]) {`
`34`	`32`	`return Q.resolve().nodeify(callback);`
`35`	`33`	`}`