Skip to content

Proper fix for 1fa30b7403176349557d6aea2f02a2075aa6f03b #4439

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 6, 2017

Conversation

sg-
Copy link
Contributor

@sg- sg- commented Jun 4, 2017

For reference

Replace #define with typedef
@0xc0170
Copy link
Contributor

0xc0170 commented Jun 5, 2017

How are these 2 files related? MBR should not use one letter words in the types, not clear what is the intention.

To the realtek file, it should be fine (not saying it's correct to have those types) as long as it does not get included to a user space (only within their driver, no exposure via header files) ?

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 5, 2017

cc @Archcady

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 5, 2017

@miklis @tommikas CI jenkins not reporting status?

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 5, 2017

@miklis @tommikas CI jenkins not reporting status?

It reported but took to long and the failure seems a glitch? Restarted

@tommikas
Copy link
Contributor

tommikas commented Jun 5, 2017

@0xc0170 The delay was because for some reason Jenkins hadn't created a job for the PR. @miklis triggered a rescan manually, which started the job.

The job then failed because one of the CI devices had had a flash failure and dropped into maintenance mode. It's back in use again so tests should be passing.

@sg-
Copy link
Contributor Author

sg- commented Jun 5, 2017

The problem is text replacement for

#define u32 uint32_t
union {
    uint32_t uint32_t
}

#define for types is no bueno IMO.

@sg-
Copy link
Contributor Author

sg- commented Jun 5, 2017

/morph test

@mbed-bot
Copy link

mbed-bot commented Jun 6, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 452

All builds and test passed!

@Archcady
Copy link
Contributor

Archcady commented Jun 6, 2017

The changes in basic_types.h looks fine, thanks.

@sg- sg- merged commit 6002bdd into ARMmbed:master Jun 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants