[CryptoModule] remove `(char *)` casting
ClosedPublic
Actions

Authored by kamil on Apr 29 2024, 9:11 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Dec 23, 4:44 PM

	Unknown Object (File)
	Mon, Dec 23, 4:44 PM

	Unknown Object (File)
	Mon, Dec 23, 4:44 PM

	Unknown Object (File)
	Mon, Dec 23, 4:44 PM

	Unknown Object (File)
	Thu, Nov 28, 5:23 PM

	Unknown Object (File)
	Nov 17 2024, 11:24 PM

	Unknown Object (File)
	Nov 12 2024, 4:26 AM

	Unknown Object (File)
	Nov 12 2024, 2:45 AM

View All 56 Files

Subscribers

ashoat

tomek

Details

Reviewers

marcin

Commits

rCOMM411a2456aa6b: [CryptoModule] remove `(char *)` casting

Summary

This addresses https://phab.comm.dev/D11717#inline-70373

Test Plan

being able to decrypt message

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Apr 29 2024, 9:11 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptApr 29 2024, 9:11 AM

kamil mentioned this in D11717: [CryptoModule] add sequential decrypt to crypto API.Apr 29 2024, 9:12 AM

kamil published this revision for review.Apr 29 2024, 9:24 AM

Harbormaster completed remote builds in B28581: Diff 39661.Apr 29 2024, 9:28 AM

marcin accepted this revision.May 6 2024, 3:37 AM

This revision is now accepted and ready to land.May 6 2024, 3:37 AM

Closed by commit rCOMM411a2456aa6b: [CryptoModule] remove `(char *)` casting (authored by kamil). · Explain WhyMay 6 2024, 4:23 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM411a2456aa6b: [CryptoModule] remove `(char *)` casting.

marcin added a reverting change: rCOMM3d71d5afcc79: Revert "[CryptoModule] remove `(char *)` casting".May 20 2024, 8:27 AM

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

CryptoTools/

Session.cpp

4 lines

Diff 39839

View Options

native/cpp/CommonCpp/CryptoTools/Session.cpp

[CryptoModule] remove `(char *)` castingClosedPublicActions