src: resolve miscellaneous conversion warnings (Windows) #10145
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
make -j8 test
(UNIX [macOS]; Linux i386 & amd64) andvcbuild test nosign
(.\vcbuild.bat nosign
then.\vcbuild.bat test nosign nobuild
as Administrator on Windows) PASS OKAdditional item
Affected core subsystem(s)
src
Description of change
Resolve other conversion warnings not covered by #10141/#10142/#10143
This change along with #10140 (build options), #10141 (URL warning fixes), #10142 (crypto warning fixes), and #10143 (dtrace warning fix) resolves all warnings on Windows if vcbuild.bat is called with the following options: noperfctr without-inspector without-profiler
NOTE: This was originally a part of PR #10139 (build, warning, header, and include fixes).
ADDITIONAL NOTE: Some of the changes are marked
// TBD POSSIBLE DATA LOSS:
since I think we should examine these conversions at some point. I would be happy to take some or all of these out if necessary.