Skip to content

Commit 8f0705d

Browse files
committed
Cleaned up old commented out code.
1 parent dae863e commit 8f0705d

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

jquery.jplayer/jquery.jplayer.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@
5252

5353
if ( isMethodCall ) {
5454
this.each(function() {
55-
// var instance = $.data( this, name ),
5655
var instance = $(this).data( name ),
5756
methodValue = instance && $.isFunction( instance[options] ) ?
5857
instance[ options ].apply( instance, args ) :
@@ -64,13 +63,11 @@
6463
});
6564
} else {
6665
this.each(function() {
67-
// var instance = $.data( this, name );
6866
var instance = $(this).data( name );
6967
if ( instance ) {
7068
// instance.option( options || {} )._init(); // Orig jquery.ui.widget.js code: Not recommend for jPlayer. ie., Applying new options to an existing instance (via the jPlayer constructor) and performing the _init(). The _init() is what concerns me. It would leave a lot of event handlers acting on jPlayer instance and the interface.
7169
instance.option( options || {} ); // The new constructor only changes the options. Changing options only has basic support atm.
7270
} else {
73-
// $.data( this, name, new $.jPlayer( options, this ) );
7471
$(this).data( name, new $.jPlayer( options, this ) );
7572
}
7673
});

0 commit comments

Comments
 (0)