search

ingydotnet / io-all-pm

All in One Perl IO
http://search.cpan.org/dist/IO-All/
38 stars 20 forks source link

subsumes method #64

Open wchristian opened 9 years ago

wchristian commented 9 years ago

Another thing from Path::Tiny, which is actually really useful to have when you need it:

https://metacpan.org/pod/Path::Tiny#subsumes

frioux commented 9 years ago

Is there a reason, other than historical, to call the method subsumes instead of contains? @dagolden

dagolden commented 9 years ago

Contains could sound like it relates to contents. I've never been fond of the name subsumes, either, so I have little advice to give, either. On Dec 13, 2014 12:04 PM, "fREW Schmidt" notifications@github.com wrote:

Is there a reason, other than historical, to call the method subsumes instead of contains? @dagolden https://github.com/dagolden

— Reply to this email directly or view it on GitHub https://github.com/ingydotnet/io-all-pm/issues/64#issuecomment-66883037.

dagolden commented 9 years ago

If you implement subsumes, you probably also want realpath and you should look at what Path::Tiny does to avoid various Cwd issues for that, too.