Paths

Table of Contentst

Differential D4820

[native] set up CXX bridge
ClosedPublic
Actions

Authored by varun on Aug 12 2022, 3:43 AM.

Tags

None

Referenced Files

None

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 12 2022, 3:43 AM

Herald added subscribers: • abosh, atul, • adrian, ashoat. · View Herald TranscriptAug 12 2022, 3:43 AM

Harbormaster completed remote builds in B11328: Diff 15588.Aug 12 2022, 3:53 AM

varun requested review of this revision.Aug 12 2022, 3:53 AM

varun added a child revision: D4821: [native] add corrosion to android cmake project.Aug 12 2022, 5:14 AM

tomek accepted this revision.Aug 12 2022, 10: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, 10:56 AM

• karol accepted this revision.Aug 12 2022, 11: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 subscribed.Aug 12 2022, 5:14 PM

• 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, 6:14 PM

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. · Explain WhyAug 18 2022, 6:34 PM

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...