Skip to content

Conversation

@Lastique
Copy link
Member

@Lastique Lastique commented May 3, 2023

This makes sizeof(tribool) == 1, which is similar to sizeof(bool) on most platforms. This is only enabled when C++11 scoped enums are supported.

@mclow
Copy link
Contributor

mclow commented May 3, 2023

Note that sizeof(bool) == 4 on most unix systems.

@Lastique
Copy link
Member Author

Lastique commented May 3, 2023

You mean legacy UNIX'es that are now dead? I'm only aware of Solaris on Sparc where this is true. On Linux and, I think, all BSDs, sizeof(bool) is 1. Windows, too.

@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Merging #28 (d53263d) into develop (1457784) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #28   +/-   ##
========================================
  Coverage    97.72%   97.72%           
========================================
  Files            2        2           
  Lines          132      132           
========================================
  Hits           129      129           
  Misses           3        3           
Impacted Files Coverage Δ
include/boost/logic/tribool.hpp 98.27% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1457784...d53263d. Read the comment docs.

@jeking3
Copy link
Collaborator

jeking3 commented Jul 25, 2025

@Lastique can you rebase on develop to get the full set of CI to run on this?

This makes sizeof(tribool) == 1, which is similar to sizeof(bool) on most
platforms. This is only enabled when C++11 scoped enums are supported.
@Lastique Lastique force-pushed the feature/reduce_tribool_size branch from d53263d to 6c61a8a Compare July 25, 2025 23:32
@Lastique
Copy link
Member Author

@Lastique can you rebase on develop to get the full set of CI to run on this?

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants