id,summary,reporter,owner,description,type,status,milestone,component,version,severity,resolution,keywords,cc
11599,"VS2015 warns that in new_tree, not all control paths return a value",Edward Brey <edward.brey@…>,Sebastian Redl,"In standard_callbacks::new_tree, Visual Studio 2015 can't tell from the assert(false) that the method will never exit after the switch statement, and so issues this warning:

warning C4715: 'boost::property_tree::json_parser::detail::standard_callbacks<boost::property_tree::basic_ptree<std::basic_string<char,std::char_traits<char>,std::allocator<char> >,std::basic_string<char,std::char_traits<char>,std::allocator<char> >,std::less<std::basic_string<char,std::char_traits<char>,std::allocator<char> > > > >::new_tree': not all control paths return a value

One solution to eliminate the warning and make the code more performant and, in case of a bug, more repeatable, is to change the end of the method to use this code:

            default:
                assert(l.k == leaf);
                stack.pop_back();
                return new_tree();
            }
        }

This warning appears for both Debug and Release builds.",Bugs,closed,To Be Determined,property_tree,Boost 1.59.0,Problem,fixed,warnings,