Comments stripped from empty functions

[acdha]

This is definitely an edge case but it might surprise people with placeholders, etc. if they aren't carefully reviewing changes:

function foo() {
	//foobar
}

becomes:

function foo() {}

Adding any sort of real work to that code will cause the comment to be retained:

function foo() {
	//foobar
        1;
}

function foo() {
  //foobar
  1;
}

[jlongster]

Good call, I think that's called a "dangling" comment and we might just not be outputting those.

[deoxxa]

I do this every now and then in empty catch handlers, like this:

doTheThing().then(something).then(somethingElse).then((res) => {
  console.log(res);
}).catch((err) => {
  /* do nothing - no way to recover from this error */
})

It makes no operational difference to the program, but it serves as documentation that the error has been considered and ignored on purpose.

[n1k0]

This comes especially annoying when iterating over some implementation and using the format-on-save feature of some editor's plugin - which turns being totally unusable in these circumstances :)

[jlongster]

Moved to #275