Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doctest: Do not use unnamed class #788

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Commits on Apr 17, 2023

  1. doctest: Do not use unnamed class

    When compiling with clang this results in emitting absolute path into
    debug info, especially .debug_str section has
    
    ~(unnamed class at /mnt/b/yoe/master/build/tmp/work/core2-64-yoe-linux-musl/libcereal/1.3.2+gitAUTOINC+ebef1e9298-r0/git/unittests/doctest.h:6428:5)
    (unnamed class at /mnt/b/yoe/master/build/tmp/work/core2-64-yoe-linux-musl/libcereal/1.3.2+gitAUTOINC+ebef1e9298-r0/git/unittests/doctest.h:6428:5)
    
    Therefore use a name for the class instead and help get rid of this
    absolute path in debug info. This fixes
    
    File /usr/lib/libcereal/ptest/tests/.debug/test_unordered_map in package libcereal-dbg contains reference to TMPDIR
    
    Signed-off-by: Khem Raj <[email protected]>
    kraj committed Apr 17, 2023
    Configuration menu
    Copy the full SHA
    2189c3a View commit details
    Browse the repository at this point in the history