Skip to content

Fix narrowing conversion warnings#11

Open
jagerman wants to merge 1 commit intosession-foundation:devfrom
jagerman:fix-narrowing-conversion
Open

Fix narrowing conversion warnings#11
jagerman wants to merge 1 commit intosession-foundation:devfrom
jagerman:fix-narrowing-conversion

Conversation

@jagerman
Copy link
Member

Fixes this warning:

libquic/src/internal.hpp: In function ‘timeval oxen::quic::loop_time_to_timeval(std::chrono::microseconds)’:
libquic/src/internal.hpp:42:34: warning: narrowing conversion of ‘std::chrono::operator/<long long int, std::ratio<1, 1000000>, long long int, std::ratio<1> >(t, std::literals::chrono_literals::operator""s<'1'>())’ from ‘time_t’ {aka ‘long long int’} to ‘long int’ [-Wnarrowing]
   42 |         return timeval{.tv_sec = static_cast<time_t>(t / 1s), .tv_usec = static_cast<suseconds_t>((t % 1s) / 1us)};
      |                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~

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.

1 participant