Skip to content

Conversation

@kant2002
Copy link
Collaborator

Let's have Cesium process two files.
First file will have only signatures, and as such it will only declaration and empty structure, since it's only declared.

Second file will have proper definition for function, and thus we should assume that types the same. Declaration part will have type with 0 members. Definition part will have type with all members declared.

Let's have Cesium process two files.
First file will have only signatures, and as such it will only declaration and empty structure, since it's only declared.

Second file will have proper definition for function, and thus we should assume that types the same.
Declaration part will have type with 0 members. Definition part will have type with all members declared.
@ForNeVeR
Copy link
Owner

A test wouldn't hurt! Generally I think the idea is sound.

@ForNeVeR
Copy link
Owner

@kant2002 do you insist on merging this as is (I'd approve) or would you like to add a test here yourself? Or delegate to me?

@kant2002
Copy link
Collaborator Author

I do not think we have in test infra what's required to properly test. I was thinking abotu using testname.c/ folder as some sort of project where we can have multiple files which consist of one app. Yeah we need this in our testing framework, but I really prefer it to be separately.

It's possible that you may combine that work with multi-arch testing, but probably it can be done separately. I prefer to work on each thing separately.

@ForNeVeR
Copy link
Owner

I do not think we have in test infra what's required to properly test.

Are you sure? We have MultiFileCompilationTest, doesn't it do kinda what you wanted here? If not, then what's missing? Let's discuss and implement!

It's possible that you may combine that work with multi-arch testing, but probably it can be done separately. I prefer to work on each thing separately.

Honestly I'd prefer to not link these two tasks so far.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants