Помощь Помогите понять, что же накодили создатели WordPress

[PHPCod3r]

Вообщем решил мальца доработать свои блог, он у меня на вордпресс, полез в код движка и по находил много интересного.
Вообщем я этой теме я бы хотел бы обсудить не ясные моменты в коде ну и то от чего можно отказаться без потери производительности блога.
Итак поехали, смотрю я функцию wp_unique_filename
и вижу в ней

while ( file_exists( $dir . "/$filename" ) ) {
			if ( '' == "$number$ext" )
				$filename = $filename . ++$number . $ext;
			else
				$filename = str_replace( "$number$ext", ++$number . $ext, $filename );
		}

Зачем while использовать если нам всего то нужно условие проверить
file_exists?
Далее "порадовал" класс File_Upload_Upgrader

if ( empty($_FILES[$form]['name']) && empty($_GET[$urlholder]) )
			wp_die(__('Please select a file'));
		if ( !empty($_FILES) )
			$this->filename = $_FILES[$form]['name'];
		else if ( isset($_GET[$urlholder]) )
			$this->filename = $_GET[$urlholder];

Зачем нам проверять

if ( empty($_FILES[$form]['name']) && empty($_GET[$urlholder]) )

если ниже проверяются более строгии условия?
Вывести предупреждения можно и если не срастется с условиями ниже, а лишнии код убрать....
Кстати лишнего кода в вордпрессе дохрена, так что если кому нужно могу выкладывать в эту тему очищенные файлы WordPress
без идиотизмов описанных выше.

 

[sinusoida]

лишнего кода в вордпрессе дохрена, так что если кому нужно могу выкладывать в эту тему очищенные файлы WordPress
без идиотизмов описанных выше.

Строка вроде "$number$ext" тоже не блещет, прямо скажем.)
Удалять лишний код в WP, как мне кажется, - борьба с ветряными мельницами. После автоматического обновления версии изменения затрутся А вот пройтись с профайлером и посмотреть какие места кода отжерают больше всего ресурсов и чего-то исправить - тут сообщество WP, уверен, скажет огромное спасибо.
Сам стараюсь в код самого движка не заглядывать по принципу "работает? не трожь", но ведь там есть места где такое нагорожено... Подобные перлы в коде часто появляются после того как находят какую-то уязвимость, а потом разработчики вместе с самой уязвимостью добавляют дополнительных проверок в других частях кода, где потенциально может что-то вылезти. "На всякий случай"