Skip to content

Conversation

@philfreo
Copy link
Contributor

removing closing php bracket to avoid extra bad whitespace afterwards (can be a problem if including this file before headers are sent)

… (can be a problem if including this file before headers are sent)
@lelutin
Copy link
Owner

lelutin commented Apr 12, 2011

hmm that's weird, I don't have a blank space at the end of the file.. I usually prefer to actually close tags but if it's problematic for you I guess it won't hurt to include your change so I'll merge it.

@philfreo
Copy link
Contributor Author

you can see the blank line (1727) here: https://github.com/lelutin/php-optparse/blob/master/optparse.php
ya - nice to remove the closing tag in classes that are pure php

@lelutin
Copy link
Owner

lelutin commented Apr 12, 2011

well, I've merged you change so I'll close this request. thanks!

@lelutin lelutin closed this Apr 12, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants