fix: extract correct function name for C++ scoped definitions

[yetf1025-dev]

Summary

• Fix C++ function name extraction for scoped definitions (ClassName::method_name) where the return type is a non-primitive type. Previously, the return type was incorrectly extracted as the function name.
• Add field_identifier to the generic name loop for C++ class member function declarations.

Problem

std::string OSDMap::get_pool_name(int64_t pool_id) const { ... }
bufferlist OSDService::get_inc_map(epoch_t e) { ... }
string RGWDedupProcessor::get_obj_fingerprint(const rgw_obj& obj) { ... }

Function	Before	After
std::string OSDMap::get_pool_name(...)	OSDMap ❌	get_pool_name ✅
bufferlist OSDService::get_inc_map(...)	bufferlist ❌	get_inc_map ✅
string RGWDedupProcessor::get_obj_fingerprint(...)	string ❌	get_obj_fingerprint ✅
int OSD::handle_osd_map(...)	handle_osd_map ✅	handle_osd_map ✅

Root Cause

_get_name recurses into function_declarator for C++, but qualified_identifier nodes (used for scoped names like Foo::bar) are not handled by the generic identifier loop. The recursion returns None and the outer loop matches the return type's type_identifier first.

Fix

Detect qualified_identifier inside function_declarator and take the rightmost identifier/field_identifier (the actual method name after the last ::).

Test plan

• 5 new regression tests in TestCppScopedFunctionName covering scoped functions with type_identifier, qualified_identifier, primitive_type return types, and unscoped functions
• All existing C++ parsing tests pass (test_multilang.py, test_parser.py, test_visualization.cpp)

Closes #395

🤖 Generated with Claude Code