string '//# sourceMappingURL=' gets removed

[thisconnect]

http://rollupjs.org/?version=0.36.0&shareable=JTdCJTIyb3B0aW9ucyUyMiUzQSU3QiUyMmZvcm1hdCUyMiUzQSUyMmNqcyUyMiUyQyUyMm1vZHVsZU5hbWUlMjIlM0ElMjJteUJ1bmRsZSUyMiUyQyUyMmdsb2JhbHMlMjIlM0ElN0IlN0QlN0QlMkMlMjJtb2R1bGVzJTIyJTNBJTVCJTdCJTIybmFtZSUyMiUzQSUyMm1haW4uanMlMjIlMkMlMjJjb2RlJTIyJTNBJTIyaW1wb3J0JTIwYWRkTWFwcGluZyUyMGZyb20lMjAnLiUyRnRlc3QuanMnJTNCJTVDbmNvbnNvbGUubG9nKCUyMGFkZE1hcHBpbmcoJTIwJ3Rlc3QnJTIwKSUyMCklM0IlNUNuY29uc29sZS5sb2coJyU1QyU1Q24lMkYlMkYlMjMlMjBzb3VyY2VNYXBwaW5nVVJMJTNEJyklMjIlN0QlMkMlN0IlMjJuYW1lJTIyJTNBJTIydGVzdC5qcyUyMiUyQyUyMmNvZGUlMjIlM0ElMjJleHBvcnQlMjBkZWZhdWx0JTIwZnVuY3Rpb24lMjBhZGRNYXBwaW5nJTIwKCUyMG9yZyUyMCklMjAlN0IlNUNuJTVDdHJldHVybiUyMG9yZyUyMCUyQiUyMCclNUMlNUNuJTJGJTJGJTIzJTIwc291cmNlTWFwcGluZ1VSTCUzRCclNUNuJTdEJTIyJTdEJTVEJTdE

[Victorystick]

See this source file for how Rollup circumvents this issue; and where it introduces it.

[thisconnect]

Do you consider this a bug?

[Victorystick]

I think it is, and the same applies to any other tools that indiscriminately remove all occurrences of that string.

[lemmabit]

Check this out. Even if it's really a comment, the regular expression has a \s in the middle of it, which will match the end of a multiline string.

[thisconnect]

thank you!

[Rich-Harris]

Released 0.38 with the fix

[lemmabit]

@Rich-Harris It's a lot better but still isn't entirely fixed. For instance, @thisconnect's original example could be rewritten like this. I'll leave it up to you whether this edge case is worth worrying about.

[Rich-Harris]

@Permutatrix yeah, I missed that – fixed by #1171