Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

visibility 'private' to visibility 'protected' #11

Merged
merged 2 commits into from Aug 17, 2011

Conversation

Projects
None yet
2 participants
Contributor

rhbecker commented Jun 28, 2011

As discussed here: #10 (comment)

Also threw in what I believe to be a bug (hopefully not me being dense) - misnamed method call from Curl library.

In case it's of interest (re: visibility) - I maintain your libraries in their own sandboxed repository, outside of my app's library directory. In my application's library directory, I then create my own library in which I require your library and extend as necessary. If anyone reading this sees a better way to accomplish what I'm doing, I'm open.

philsturgeon pushed a commit that referenced this pull request Aug 17, 2011

Merge pull request #11 from rhbecker/master
visibility 'private' to visibility 'protected'

@philsturgeon philsturgeon merged commit 1b0c018 into philsturgeon:master Aug 17, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment