Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Feature module start time#311

Merged
konglobemeralt merged 42 commits into
devfrom
feature_moduleStartTime
Jan 26, 2021
Merged

Feature module start time#311
konglobemeralt merged 42 commits into
devfrom
feature_moduleStartTime

Conversation

@konglobemeralt

Copy link
Copy Markdown
Member

Wait until we have a way to deal with the extended buffer before merging. Each module now enters the proc file of its PID and returns its start time when receiving GET_STATUS

@LukasWikander LukasWikander left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm guessing jenkins has the same issue here as for the other ones...

Comment thread core/src/systemcontrol.c Outdated
Comment thread modules/ScenarioControl/src/main.cpp Outdated
Comment thread modules/ScenarioControl/src/main.cpp Outdated
Comment thread modules/ScenarioControl/src/main.cpp Outdated
Comment thread modules/ScenarioControl/src/main.cpp Outdated
Comment thread modules/ScenarioControl/src/main.cpp Outdated
Comment thread modules/ScenarioControl/src/main.cpp Outdated
Comment thread modules/Supervision/src/main.cpp Outdated
Comment thread core/src/util.c Outdated
Comment thread core/src/util.c Outdated
Comment thread core/src/util.c Outdated
Comment thread core/src/systemcontrol.c Outdated

@LukasWikander LukasWikander left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just minor stuff left. Also, you may want to run the code formatter.

Comment thread core/src/util.c Outdated
Comment thread core/inc/util.h Outdated
Comment thread core/src/systemcontrol.c Outdated
Comment thread core/src/util.c Outdated
@konglobemeralt konglobemeralt merged commit b2889f7 into dev Jan 26, 2021
@konglobemeralt konglobemeralt deleted the feature_moduleStartTime branch January 26, 2021 09:03
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