[CommCoreModule] Use braced initialization in `CryptoModule`
ClosedPublic
Actions

Authored by atul on Feb 23 2022, 10:00 AM.

Details

Reviewers

• karol
varun
tomek
• jim
ashoat

Commits

rCOMMa6a83b0cb1c0: [CommCoreModule] Use braced initialization in `CryptoModule`

Summary

noop, just noticed while working in CryptoModule and took 2 seconds to fix

see: https://www.notion.so/commapp/C-coding-standards-8c9f22b4b16a41c3a868eb5f537db1de#7de58b7ac3c0402bbae2a4e3b6b6738c for context

Test Plan

Continues to build

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Feb 23 2022, 10:00 AM

Herald added subscribers: • adrian, ashoat. · View Herald TranscriptFeb 23 2022, 10:00 AM

Harbormaster completed remote builds in B6978: Diff 9842.Feb 23 2022, 10:05 AM

atul requested review of this revision.Feb 23 2022, 10:05 AM

tomek accepted this revision.Feb 23 2022, 10:26 AM

This revision is now accepted and ready to land.Feb 23 2022, 10:26 AM

Herald added a reviewer: ashoat. · View Herald TranscriptFeb 23 2022, 10:26 AM

Herald added a subscriber: • benschac. · View Herald Transcript

This revision now requires review to proceed.Feb 23 2022, 10:26 AM

ashoat accepted this revision.Feb 23 2022, 11:45 AM

This revision is now accepted and ready to land.Feb 23 2022, 11:45 AM

Closed by commit rCOMMa6a83b0cb1c0: [CommCoreModule] Use braced initialization in `CryptoModule` (authored by atul). · Explain WhyFeb 23 2022, 12:16 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMa6a83b0cb1c0: [CommCoreModule] Use braced initialization in `CryptoModule`.

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

CryptoTools/

CryptoModule.cpp

58 lines

Diff 9847

View Options

native/cpp/CommonCpp/CryptoTools/CryptoModule.cpp

[CommCoreModule] Use braced initialization in `CryptoModule`ClosedPublicActions