Skip to content

Commit 50a1545

Browse files
committed
Some more fixes
1 parent 6550dfd commit 50a1545

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/Events/php_exporter.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ public function get_event_name($event_line, $is_dispatch)
195195
preg_match(sprintf($regex, $this->preg_match_event_name_uppercase()), $event_text_line, $match);
196196

197197
if (isset($match[2]))
198-
{;
198+
{
199199
$this->output->addMessage(Output::ERROR, sprintf('Event names should be all lowercase in %s for event %s', $this->current_file, $match[2]));
200200
}
201201
else

src/Files/FileLoader.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public function loadFile($fileName)
6363
// Otherwise add notice.
6464
if (strtolower($fileName) !== 'readme')
6565
{
66-
$this->output->addMessage(Output::NOTICE, sprintf("The file %s has no valid extension.", basename($fileName)), null);
66+
$this->output->addMessage(Output::NOTICE, sprintf("The file %s has no valid extension.", basename($fileName)));
6767
}
6868
$file = new PlainFile($this->debug, $fileName);
6969
}
@@ -90,7 +90,7 @@ public function loadFile($fileName)
9090
else if ($size >= 4) // Files with 3 ore more dots should not happen.
9191
{
9292
$error = true;
93-
$this->output->addMessage(Output::ERROR, sprintf("File (%s) seems to have too many dots. Using the last part as extension.", $fileName), null);
93+
$this->output->addMessage(Output::ERROR, sprintf("File (%s) seems to have too many dots. Using the last part as extension.", $fileName));
9494

9595
$file = self::tryLoadFile($fileName, $split[sizeof($split) - 1]);
9696

@@ -193,11 +193,11 @@ private function tryLoadFile($fileName, $extension, $returnNull = false)
193193
return new ImageFile($this->debug, $fileName);
194194

195195
case 'swf':
196-
$this->output->addMessage(Output::NOTICE, sprintf("Found an SWF file (%s), please make sure to include the source files for it, as required by the GPL.", basename($fileName)), null);
196+
$this->output->addMessage(Output::NOTICE, sprintf("Found an SWF file (%s), please make sure to include the source files for it, as required by the GPL.", basename($fileName)));
197197

198198
return new BinaryFile($this->debug, $fileName);
199199
case 'ds_store':
200-
$this->output->addMessage(Output::ERROR, sprintf("Found an OS X specific file at %s, please make sure to remove it prior to submission.", $fileName), null);
200+
$this->output->addMessage(Output::ERROR, sprintf("Found an OS X specific file at %s, please make sure to remove it prior to submission.", $fileName));
201201

202202
return new BinaryFile($this->debug, $fileName);
203203

0 commit comments

Comments
 (0)