You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
performance is somewhat a concern since we now need to also store indices in the token list.
perhaps the better way to keep this optional and out of the core is to optionally return a list of indices alongside the token list with indications of which were removed and leave the actual work of doing the sourcemapping to an external wrapper instead of bloating the core.
or better yet? add the location index as arg to shouldDrop(sel, loc) and let the consumer/wrapper accumulate the deleted array. (still behind an option flag).
The text was updated successfully, but these errors were encountered:
i've personally not needed it, but it could be a quality of life improvement.
probably via https://github.com/Rich-Harris/magic-string but not sure how useful it would be without also ingesting a prior sourcemap, so also https://github.com/Rich-Harris/sorcery.
performance is somewhat a concern since we now need to also store indices in the token list.
perhaps the better way to keep this optional and out of the core is to optionally return a list of indices alongside the token list with indications of which were removed and leave the actual work of doing the sourcemapping to an external wrapper instead of bloating the core.
or better yet? add the location index as arg to
shouldDrop(sel, loc)
and let the consumer/wrapper accumulate the deleted array. (still behind an option flag).The text was updated successfully, but these errors were encountered: