Skip to content

Raise limit on code size before compacting#4965

Merged
hzoo merged 1 commit intobabel:masterfrom
existentialism:issue4957
Dec 8, 2016
Merged

Raise limit on code size before compacting#4965
hzoo merged 1 commit intobabel:masterfrom
existentialism:issue4957

Conversation

@existentialism
Copy link
Copy Markdown
Member

Q A
Bug fix? N
Breaking change? N
New feature? N
Deprecations? N
Spec compliancy? N
Tests added/pass? Y
Fixed tickets Fixes #4957
License MIT
Doc PR
Dependency Changes N

I know @loganfsmyth has mentioned 500kb a few times, figured we can remove it altogether after some time has passed.

@codecov-io
Copy link
Copy Markdown

Current coverage is 89.23% (diff: 50.00%)

Merging #4965 into master will not change coverage

@@             master      #4965   diff @@
==========================================
  Files           204        204          
  Lines         10494      10494          
  Methods        1154       1154          
  Messages          0          0          
  Branches       2761       2761          
==========================================
  Hits           9364       9364          
  Misses         1130       1130          
  Partials          0          0          

Powered by Codecov. Last update 26b4e09...b0f4477

@hzoo hzoo added the PR: Polish 💅 A type of pull request used for our changelog categories label Dec 8, 2016
@DrewML
Copy link
Copy Markdown
Member

DrewML commented Dec 8, 2016

We should remember to update the docs here, too.

Do not include superfluous whitespace characters and line terminators. When set to "auto" compact is set to true on input sizes of >100KB.

@hzoo hzoo merged commit db04f99 into babel:master Dec 8, 2016
@existentialism existentialism deleted the issue4957 branch December 8, 2016 23:15
Jessidhia pushed a commit to Jessidhia/babel that referenced this pull request Dec 12, 2016
* master:
  make installing runtime/transform-runtime clearer [skip ci] (babel#4991)
  Add example to es2015-unicode-regex [skip ci] (babel#4983)
  v6.20.3
  Calculate the correct arity for async functions with destructuring - fixes babel#4977 (babel#4978)
  v6.20.2
  fix object spread (babel#4976)
  fix clean lib
  update readme [skip ci]
  v6.20.1
  Fix nested object spread (babel#4974)
  v6.20.0
  v6.20.0 changelog [skip ci] (babel#4971)
  Raise limit on code size before compacting (babel#4965)
  Use regenerator-transform to implement babel-plugin-transform-regenerator (babel#4881)
  Add getBindingIdentifierPaths/getOuterBindingIdentifierPaths (babel#4876)
  Hoist generateDeclaredUidIdentifier helper function (babel#4934)
Jessidhia pushed a commit to Jessidhia/babel that referenced this pull request Dec 16, 2016
* master: (89 commits)
  fix sizing [skip ci]
  typo [skip ci]
  changelog for v6.20.1 to v6.20.3 [skip ci]
  mention repl/link [skip ci]
  add semver label [skip ci]
  Fix links in CONTRIBUTING.md [skip ci] (babel#4989)
  make installing runtime/transform-runtime clearer [skip ci] (babel#4991)
  Add example to es2015-unicode-regex [skip ci] (babel#4983)
  v6.20.3
  Calculate the correct arity for async functions with destructuring - fixes babel#4977 (babel#4978)
  v6.20.2
  fix object spread (babel#4976)
  fix clean lib
  update readme [skip ci]
  v6.20.1
  Fix nested object spread (babel#4974)
  v6.20.0
  v6.20.0 changelog [skip ci] (babel#4971)
  Raise limit on code size before compacting (babel#4965)
  Use regenerator-transform to implement babel-plugin-transform-regenerator (babel#4881)
  ...
@boujeepossum
Copy link
Copy Markdown
Contributor

boujeepossum commented Jan 9, 2017

this change resulted in some issues in one of my builds.
I have a relatively large file that i'm instrumenting with babel-plugin-istanbul(length 141415)

with this limit being raised to 500000 my babel process gets stuck on 100% and never exits.

i was able to track it and the last babel line of code that gets frozen is this https://github.com/babel/babel/blob/master/packages/babel-generator/src/buffer.js#L43

after that it leaves babel and goes into lodash code (https://github.com/lodash/lodash/blob/master/trimEnd.js#L32)
where the process gets stuck on string.replace() call that seems to be not able to process such a big string (string length here is 6449484)

reverting this change solves the issue. Is there any way to configure it or can it be reverted?

@DrewML
Copy link
Copy Markdown
Member

DrewML commented Jan 10, 2017

@DmitriiAbramov Opened a separate issue (#5081) to discuss this further so this comment doesn't get lost in the shuffle.

panagosg7 pushed a commit to panagosg7/babel that referenced this pull request Jan 17, 2017
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 6, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Polish 💅 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove/Change the limit of the babel-generator deopt message

5 participants