Skip to content
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

Events per signal instance #367

Merged
merged 12 commits into from
Nov 29, 2019
Merged

Events per signal instance #367

merged 12 commits into from
Nov 29, 2019

Conversation

wegendt-bosch
Copy link
Contributor

@wegendt-bosch wegendt-bosch commented Nov 18, 2019

Based on #344.

@wegendt-bosch wegendt-bosch force-pushed the events-per-signal-instance branch from 1899a41 to ecbb9f2 Compare November 18, 2019 15:17
Copy link

@csweichel csweichel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment on lines -185 to -187
if(tv.idx == 2679 && typ.toString == "uint32") {
print("")
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Always happy to see those go :)

@@ -219,7 +219,7 @@ class RingbufferGenerator extends AbstractTypeGenerator {
),
true
«rbRef.code».length++;
++«rbRef.code».length;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this just a readability issue or did this produce compilation errors?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general, preincrement > postincrement, since the latter generates a temporary copy ;)

}

override getLibraries() {
return #["paho-mqtt3c"]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we ping the Paho guys as Mita now has Paho on x86 support? :D

@@ -73,11 +73,11 @@ class EventLoopGenerator implements IPlatformEventLoopGenerator {
}

override generateSetupPreamble(CompilationContext context) {
val startupEventHandler = context.allEventHandlers.filter[it.event instanceof SystemEventSource].findFirst[(it.event as SystemEventSource).source.name == "startup"]
val startupEventHandlers = context.allEventHandlers.filter[it.event instanceof SystemEventSource].filter[(it.event as SystemEventSource).source.name == "startup"]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do resources other than the platform allow startup handler, too?

Copy link
Contributor Author

@wegendt-bosch wegendt-bosch Nov 29, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you might have more than one startup handler for the event

@wegendt-bosch wegendt-bosch force-pushed the events-per-signal-instance branch from 2509f97 to 5abdb17 Compare November 29, 2019 16:17
@wegendt-bosch wegendt-bosch merged commit bccedda into master Nov 29, 2019
PriyaGovind pushed a commit to PriyaGovind/mita that referenced this pull request Apr 15, 2020
* Implement events per signal instance
* Implement user specified event handler buffer sizes
* Implement MQTT on x86
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.

2 participants