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

Add JLSE CMake preset and a few small fixes #1611

Merged
merged 5 commits into from
Feb 5, 2025

Conversation

amandalund
Copy link
Contributor

No description provided.

In file included from /soft/compilers/gcc/12.2.0/x86_64-suse-linux/include/c++/12.2.0/string:40,
                 from /home/alund/celeritas_project/celeritas/src/corecel/io/Label.hh:12,
                 from /home/alund/celeritas_project/celeritas/test/corecel/io/Label.test.cc:7:
In static member function ‘static constexpr int std::char_traits<char>::compare(const char_type*, const char_type*, std::size_t)’,
    inlined from ‘int std::__cxx11::basic_string<_CharT, _Traits, _Alloc>::compare(const std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&) const [with _CharT = char; _Traits = std::char_traits<char>; _Alloc = std::allocator<char>]’ at /soft/compilers/gcc/12.2.0/x86_64-suse-linux/include/c++/12.2.0/bits/basic_string.h:3148:32,
    inlined from ‘bool std::operator<(const __cxx11::basic_string<_CharT, _Traits, _Allocator>&, const __cxx11::basic_string<_CharT, _Traits, _Allocator>&) [with _CharT = char; _Traits = char_traits<char>; _Alloc = allocator<char>]’ at /soft/compilers/gcc/12.2.0/x86_64-suse-linux/include/c++/12.2.0/bits/basic_string.h:3694:27,
    inlined from ‘bool celeritas::operator<(const Label&, const Label&)’ at /home/alund/celeritas_project/celeritas/src/corecel/io/Label.hh:98:17,
    inlined from ‘bool celeritas::operator<(const Label&, const Label&)’ at /home/alund/celeritas_project/celeritas/src/corecel/io/Label.hh:92:13,
    inlined from ‘virtual void celeritas::test::LabelTest_ordering_Test::TestBody()’ at /home/alund/celeritas_project/celeritas/test/corecel/io/Label.test.cc:28:5:
/soft/compilers/gcc/12.2.0/x86_64-suse-linux/include/c++/12.2.0/bits/char_traits.h:385:32: error: ‘*(const unsigned char*)((char*)&<unnamed> + offsetof(celeritas::Label, celeritas::Label::ext.std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::<unnamed>))’ may be used uninitialized [-Werror=maybe-uninitialized]
  385 |         return __builtin_memcmp(__s1, __s2, __n);
      |                ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
In file included from /home/alund/spack/var/spack/environments/celeritas/.spack-env/view/include/gtest/gtest-printers.h:122,
                 from /home/alund/spack/var/spack/environments/celeritas/.spack-env/view/include/gtest/gtest-matchers.h:49,
                 from /home/alund/spack/var/spack/environments/celeritas/.spack-env/view/include/gtest/internal/gtest-death-test-internal.h:47,
                 from /home/alund/spack/var/spack/environments/celeritas/.spack-env/view/include/gtest/gtest-death-test.h:43,
                 from /home/alund/spack/var/spack/environments/celeritas/.spack-env/view/include/gtest/gtest.h:64,
                 from /home/alund/celeritas_project/celeritas/test/Test.hh:12,
                 from /home/alund/celeritas_project/celeritas/test/celeritas_test.hh:16,
                 from /home/alund/celeritas_project/celeritas/test/corecel/io/Label.test.cc:9:
/home/alund/celeritas_project/celeritas/test/corecel/io/Label.test.cc: In member function ‘virtual void celeritas::test::LabelTest_ordering_Test::TestBody()’:
/home/alund/celeritas_project/celeritas/test/corecel/io/Label.test.cc:28:50: note: ‘<anonymous>’ declared here
   28 |     EXPECT_FALSE(Label("a", "2") < Label("a", "1"));
      |                                                  ^
@amandalund amandalund added documentation Documentation, examples, tests, and CI minor Minor internal changes or fixes labels Feb 5, 2025
@amandalund amandalund requested a review from sethrj February 5, 2025 18:40
Copy link
Member

@sethrj sethrj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

src/corecel/io/Label.hh Show resolved Hide resolved
Copy link

github-actions bot commented Feb 5, 2025

Test summary

 1 953 files   3 069 suites   1m 6s ⏱️
 1 119 tests  1 108 ✅ 11 💤 0 ❌
10 116 runs  10 088 ✅ 28 💤 0 ❌

Results for commit b30d740.

@sethrj sethrj enabled auto-merge (squash) February 5, 2025 19:01
@sethrj sethrj merged commit b607c51 into celeritas-project:develop Feb 5, 2025
36 checks passed
@amandalund amandalund deleted the minor-fixes branch February 5, 2025 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation, examples, tests, and CI minor Minor internal changes or fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants