Remove unused code in media.php

[arena]

linkifunknown variable is useless because always false when tested !

wp-includes/media.php:1029:	var $linkifunknown = true;
wp-includes/media.php:1276:		$oldval = $this->linkifunknown;
wp-includes/media.php:1277:		$this->linkifunknown = false;
wp-includes/media.php:1279:		$this->linkifunknown = $oldval;
wp-includes/media.php:1291:		$output = ( $this->linkifunknown ) ? '<a href="' . esc_attr($url) . '">' . esc_html($url) . '</a>' : $url;

[kurtpayne]

It's not immediately clear what's going on, but it's definitely not dead code.

It's first declared as true:

var $linkifunknown = true;

Then a shortcode named 'embed' is added:

add_shortcode( 'embed', array(&$this, 'shortcode') );

In the shortcode method, a maybe_make_link method is called, which uses $linkifunknown. It's true here.

$output = ( $this->linkifunknown ) ? '<a href="' . esc_attr($url) . '">' . esc_html($url) . '</a>' : $url;

$linkifunknown is only false when called from autoembed_callback.

function autoembed_callback( $match ) {
	$oldval = $this->linkifunknown;
	$this->linkifunknown = false;
	$return = $this->shortcode( array(), $match[1] );
	$this->linkifunknown = $oldval;
	return "\n$return\n";
}

When shortcode is called from autoembed_callback, $linkifunknown is false.