[native] remove comment from `BackupHandler`
ClosedPublic
Actions

Authored by kamil on Dec 3 2024, 6:33 AM.

Details

Reviewers

bartek
tomek

Commits

rCOMM066e576587e0: [native] remove comment from `BackupHandler`

Summary

I accidentally landed D14060 with a leftover, comment I added for myself when developing.

Test Plan

N/A

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Dec 3 2024, 6:33 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptDec 3 2024, 6:33 AM

kamil published this revision for review.Dec 3 2024, 6:34 AM

kamil retitled this revision from [native] remove comment from to [native] remove comment from `BackupHandler`.

Harbormaster completed remote builds in B32923: Diff 46206.Dec 3 2024, 6:51 AM

bartek accepted this revision.Dec 3 2024, 9:47 AM

This revision is now accepted and ready to land.Dec 3 2024, 9:47 AM

Closed by commit rCOMM066e576587e0: [native] remove comment from `BackupHandler` (authored by kamil). · Explain WhyDec 4 2024, 4:38 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM066e576587e0: [native] remove comment from `BackupHandler`.

Revision Contents
Changeset List

Path

Size

native/

backup/

backup-handler.js

1 line

Diff 46207

View Options

native/backup/backup-handler.js

	Show All 39 Lines
	pub enum Messages {			pub enum Messages {
	ConnectionInitializationMessage(ConnectionInitializationMessage),			ConnectionInitializationMessage(ConnectionInitializationMessage),
	ConnectionInitializationResponse(ConnectionInitializationResponse),			ConnectionInitializationResponse(ConnectionInitializationResponse),
	AnonymousInitializationMessage(AnonymousInitializationMessage),			AnonymousInitializationMessage(AnonymousInitializationMessage),
	// MessageToDeviceRequestStatus must be placed before MessageToDeviceRequest.			// MessageToDeviceRequestStatus must be placed before MessageToDeviceRequest.
	// This is due to serde's pattern matching behavior where it prioritizes			// This is due to serde's pattern matching behavior where it prioritizes
	// the first matching pattern it encounters.			// the first matching pattern it encounters.
	APNsNotif(APNsNotif),			APNsNotif(APNsNotif),
				FCMNotif(FCMNotif),
	MessageToDeviceRequestStatus(MessageToDeviceRequestStatus),			MessageToDeviceRequestStatus(MessageToDeviceRequestStatus),
	MessageToDeviceRequest(MessageToDeviceRequest),			MessageToDeviceRequest(MessageToDeviceRequest),
	MessageToDevice(MessageToDevice),			MessageToDevice(MessageToDevice),
	MessageReceiveConfirmation(MessageReceiveConfirmation),			MessageReceiveConfirmation(MessageReceiveConfirmation),
	MessageToTunnelbrokerRequest(MessageToTunnelbrokerRequest),			MessageToTunnelbrokerRequest(MessageToTunnelbrokerRequest),
	Heartbeat(Heartbeat),			Heartbeat(Heartbeat),
	IdentityDeviceListUpdated(IdentityDeviceListUpdated),			IdentityDeviceListUpdated(IdentityDeviceListUpdated),
	}			}
	Show All 13 Lines

[native] remove comment from `BackupHandler`ClosedPublicActions