Paths

Table of Contentst

Differential D4820

[native] set up CXX bridge
ClosedPublic
Actions

Authored by varun on Aug 11 2022, 8:43 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jul 3, 8:02 AM

	Unknown Object (File)
	Mon, Jul 1, 10:59 AM

	Unknown Object (File)
	Mon, Jul 1, 4:43 AM

	Unknown Object (File)
	Wed, Jun 26, 3:58 PM

	Unknown Object (File)
	Wed, Jun 26, 2:30 AM

	Unknown Object (File)
	Wed, Jun 26, 2:30 AM

	Unknown Object (File)
	Wed, Jun 26, 2:29 AM

	Unknown Object (File)
	Wed, Jun 26, 2:26 AM

View All 72 Files

Subscribers

Details

Reviewers

tomek
• karol

Commits

rCOMM3aecb610154d: [native] set up CXX bridge

Summary

create the ffi module that will be used to generate the bridge between C++ and Rust code

Depends on D4815

Test Plan

cargo build

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Aug 11 2022, 8:43 PM

Herald added subscribers: • abosh, atul, • adrian, ashoat. · View Herald TranscriptAug 11 2022, 8:43 PM

Harbormaster completed remote builds in B11328: Diff 15588.Aug 11 2022, 8:53 PM

varun requested review of this revision.Aug 11 2022, 8:53 PM

varun added a child revision: D4821: [native] add corrosion to android cmake project.Aug 11 2022, 10:14 PM

tomek accepted this revision.Aug 12 2022, 3:56 AM

tomek added inline comments.

native/cpp/CommonCpp/grpc/grpc_client/build.rs
5 ↗	(On Diff #15588)	What os the purpose of this print?

This revision is now accepted and ready to land.Aug 12 2022, 3:56 AM

• karol accepted this revision.Aug 12 2022, 4:52 AM

• karol added inline comments.

native/cpp/CommonCpp/grpc/grpc_client/src/lib.rs
51 ↗	(On Diff #15588)	What does `ffi` stand for? Can we change this name? It's not very descriptive.

https://en.wikipedia.org/wiki/Foreign_function_interface

• jon added a subscriber: • jon.Aug 12 2022, 10:14 AM

• jon added inline comments.

native/cpp/CommonCpp/grpc/grpc_client/src/lib.rs
51 ↗	(On Diff #15588)	`ffi` is a pretty common across many languages, generally the assumption is that they "act like C", and that ABI can be used across many boundries.

varun added inline comments.Aug 12 2022, 11:14 AM

native/cpp/CommonCpp/grpc/grpc_client/build.rs
5 ↗	(On Diff #15588)	It tells Cargo that if the given file changes, to rerun this build script
native/cpp/CommonCpp/grpc/grpc_client/src/lib.rs
51 ↗	(On Diff #15588)	I'd prefer to keep it as is

Closed by commit rCOMM3aecb610154d: [native] set up CXX bridge (authored by varun). · Explain WhyAug 18 2022, 11:34 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM3aecb610154d: [native] set up CXX bridge.

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

grpc/

grpc_client/

2 lines

src/

3 lines

Diff 15754

native/cpp/CommonCpp/grpc/grpc_client/build.rs

Loading...

native/cpp/CommonCpp/grpc/grpc_client/src/lib.rs

Loading...