search

boostorg / property_tree

Boost.org property_tree module
http://boost.org/libs/property_tree
55 stars 92 forks source link

Make operator / a friend of string_path #119

Closed ashtum closed 9 months ago

ashtum commented 9 months ago

@pdimov, Upon further consideration, I made the operator/ overload a friend of string_path so that it can be implicitly constructed from string and char types when needed. With that, we don't need multiple overloads (the first commit in this PR). Is this the preferred approach?

pdimov commented 9 months ago

What are we trying to fix here? https://github.com/boostorg/property_tree/pull/24 and http://trac.cpp.al/trac10/ticket/12466?

ashtum commented 9 months ago

What are we trying to fix here? #24 and http://trac.cpp.al/trac10/ticket/12466?

Yes.

pdimov commented 9 months ago

This is the preferred approach, yes. But it looks like we are missing a test for p1 / p2 with p1 and p2 both being paths.