ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-dev: Re: [Wireshark-dev] Who introduced these failures?

From: Richard Sharpe <realrichardsharpe@xxxxxxxxx>
Date: Wed, 21 Apr 2021 10:45:53 -0700
On Wed, Apr 21, 2021 at 10:47 AM Pascal Quantin <pascal@xxxxxxxxxxxxx> wrote:
>
> Hi Richard,
>
> Le mer. 21 avr. 2021 à 19:43, Richard Sharpe <realrichardsharpe@xxxxxxxxx> a écrit :
>>
>> My latest MR failed with these errors:
>>
>> -----------------------------
>> C:\builds\wireshark\wireshark\epan\dissectors\packet-componentstatus.c(172,71):
>> error C2220: workload = 100.0 *
>> CSR_GET_WORKLOAD(tvb_get_ntohs(message_tvb, 284));
>> [C:\builds\wireshark\wireshark\build\epan\dissectors\dissectors.vcxproj]
>> 857C:\builds\wireshark\wireshark\epan\dissectors\packet-componentstatus.c(172,71):
>> error C2220: ^ [C:\builds\wireshark\wireshark\build\epan\dissectors\dissectors.vcxproj]
>> 858 packet-cp2179.c
>> 859C:\builds\wireshark\wireshark\epan\dissectors\packet-componentstatus.c(172,71):
>> warning C4244: '=': conversion from 'double' to 'float', possible loss
>> of data [C:\builds\wireshark\wireshark\build\epan\dissectors\dissectors.vcxproj]
>> -------------------------------
>>
>> They are not in packet-ieee80211.c where my changes were.
>
>
> This seems related to https://gitlab.com/wireshark/wireshark/-/merge_requests/2178 where no Windows build was run by the CI, and thus the error was left unnoticed prior to the merge. See the thread initiated by Anders earlier today.

Ahhh, thanks.


-- 
Regards,
Richard Sharpe
(何以解憂?唯有杜康。--曹操)(传说杜康是酒的发明者)