Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add test for passing/getting packed structs with ffi.
- Loading branch information
Showing
3 changed files
with
46 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
-include ../tools.mk | ||
|
||
all: | ||
$(CC) -std=c99 test.c -c -o $(TMPDIR)/test.o | ||
$(AR) rcs $(TMPDIR)/libtest.a $(TMPDIR)/test.o | ||
$(RUSTC) test.rs -L $(TMPDIR) | ||
$(call RUN,test) || exit 1 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
struct __attribute__((packed)) Foo { | ||
char a; | ||
short b; | ||
char c; | ||
}; | ||
|
||
struct Foo foo(struct Foo foo) { | ||
return foo; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
// Copyright 2014 The Rust Project Developers. See the COPYRIGHT | ||
// file at the top-level directory of this distribution and at | ||
// http://rust-lang.org/COPYRIGHT. | ||
// | ||
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or | ||
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license | ||
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your | ||
// option. This file may not be copied, modified, or distributed | ||
// except according to those terms. | ||
|
||
#[packed] | ||
#[deriving(PartialEq, Show)] | ||
struct Foo { | ||
a: i8, | ||
b: i16, | ||
c: i8 | ||
} | ||
|
||
#[link(name = "test", kind = "static")] | ||
extern { | ||
fn foo(f: Foo) -> Foo; | ||
} | ||
|
||
fn main() { | ||
unsafe { | ||
let a = Foo { a: 1, b: 2, c: 3 }; | ||
let b = foo(a); | ||
assert_eq!(a, b); | ||
} | ||
} |
b90e2d4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from alexcrichton
at luqmana@b90e2d4
b90e2d4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging luqmana/rust/psfo = b90e2d4 into auto
b90e2d4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
luqmana/rust/psfo = b90e2d4 merged ok, testing candidate = a01a3692
b90e2d4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/908
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/912
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/912
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/912
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/907
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/907
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/913
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/907
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/910
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/908
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/909