Namespace Hack, CLASSES Config Option

[wbraswell]

This first commit implements the proposed solution B for the Namespace Hack, Part 2 issue: #7

Included in this commit is the code required to implement the solution, test it in the exact manner shown for the previously-existing t/namespace tests, and document it in the POD file.

There is no code included in this commit that is not directly related to, and required by, the proposed solution B.

[daoswald]

Thanks for the pull request. I'll review and hopefully merge in a few days (AFK starting in a few mins)

[wbraswell]

This second commit standardizes our use of Inline config options to be lower case, now that Inline supports case insensitive config options. All tests pass.

This second commit does not contain any changes other than standardizing on lower case config options.

[wbraswell]

This third commit implements a combination of the proposed solution C and @mohawk2's coderef proposed solution for the Namespace Hack, Part 2 issue: #7

This commit provides an automated solution which builds upon, but does not replace, the first commit implementing the manual solution A. The manual solution may be used for simple codebases, while the automated solution is needed for more complex situations, such as RPerl.

Included in this commit is the code required to implement the solution, test it in the exact manner shown for the previously-existing t/classes tests, and document it in the POD file.

There is no code included in this commit that is not directly related to, and required by, the proposed solution C and coderef combination.

[mohawk2]

That t/classes/07conflict_avoid.t needs work. If you want to create a temporary file or dir, that's fine, just use File::Temp->newdir which will auto cleanup at END.

[mohawk2]

Why are you having the code-ref return a list, not just a Perl classname?

[wbraswell]

Inline::CPP requires $pkg and $class be 2 separate variables, thus they are returned as such by the coderef solution.

[daoswald]

I've merged this into the upstream repo under the wbraswell-classes branch for further review, with the intent of being able to merge with master when ready.

Auto-merge wasn't possible since the pull request created a conflict in version numbering. But the conflict has been resolved and the pull request now exists in this repo under wbraswell-classes.

[mohawk2]

I fixed the left-in Data::Dumper line in bb4f95e on that branch.

[daoswald]

This pull request has now been adapted and merged into master, and will be released as CPAN v0.57. Thanks!

[wbraswell]

Great, thanks everyone!