Babel warning during build

[davemevans]

When building the code, the following warning is displayed:

[BABEL] Note: The code generator has deoptimised the styling of "D:/nginx/html/dash.js-2.0/src/streaming/MediaPlayer.js" as it exceeds the max of "100KB".

This seems to be related to the compact option which we don't set so defaults to auto, which causes this warning because MediaPlayer.js is enormous.

It's not doing any harm but we should probably make a decision to set compact explicitly to either true or false to make the warning go away.

[nicosang]

Hi @bbcrddave ,

you're right. It should also be interesting to decrease the size of this file.

Nico

[davemevans]

Sadly this isn't fixed as the compact option is specified on the wrong build target - it's the browserify target that needs compact set:

Replacing transform: ['babelify'] with transform: [['babelify', {compact: true}]] should do the trick (or possibly false - see #2255 for discussion about this).

[nicosang]

Hi @bbcrddave ,

after some tests, compact sould be set to false instead. Indeed, with true value, the source map seems not good. For instance, I put a break point in PlaybackController:onBytesAppended, when a seek command is called, the step into will not reached the seek code.

According to @jeffcunat , the new version (6.2) of babelify increases the limit from 100 KB to 500KB....maybe a better solution....

Nico

[davemevans]

Yes, I wondered if that may the case - see my comment at #2255 (comment).

Switching to babel 6 might be an option but I suspect it might be a big task.

[jeoliva]

Agree with @bbcrddave. As suggested by him in #2255 I think it is safer continue using current babel version and set compact to false.

[davemevans]

Should be closed via 30684b3