Skip to content

Enforce power-of-two for history setting in CMakeLists (backport #271) - #272

Merged
pablogs9 merged 1 commit into
mainfrom
mergify/bp/main/pr-271
Dec 12, 2022
Merged

Enforce power-of-two for history setting in CMakeLists (backport #271)#272
pablogs9 merged 1 commit into
mainfrom
mergify/bp/main/pr-271

Conversation

@mergify

@mergify mergify Bot commented Dec 12, 2022

Copy link
Copy Markdown
Contributor

This is an automatic backport of pull request #271 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* STREAM_HISTORY option requires power-of-two value

Signed-off-by: gavanderhoorn <g.a.vanderhoorn@tudelft.nl>

* Also mention power-of-two requirement in readme

Signed-off-by: gavanderhoorn <g.a.vanderhoorn@tudelft.nl>

* cmake: enforce power-of-two for history setting

Signed-off-by: gavanderhoorn <g.a.vanderhoorn@tudelft.nl>

Signed-off-by: gavanderhoorn <g.a.vanderhoorn@tudelft.nl>
(cherry picked from commit fb2dca6)
@pablogs9
pablogs9 force-pushed the mergify/bp/main/pr-271 branch from 7f150b7 to 49aba35 Compare December 12, 2022 11:45
@github-actions

Copy link
Copy Markdown

Static memory analysis

Default configuration

MTU: 512 B
Input buffer size: 2048 B
Input history: 4
Output buffer size: 2048 B
Output history: 4

Entity Qty Size per unit
Context 2 5616 B
Topic 8 56 B
Service 4 248 B
Client 4 248 B
Subscription 4 280 B
Publisher 4 288 B
Node 4 208 B
Static input buffer 8 2136 B
Init options 6 64 B
Wait sets 4 56 B
Guard Condition 4 64 B

TOTAL: 34500 B

@pablogs9
pablogs9 merged commit a08c301 into main Dec 12, 2022
@pablogs9
pablogs9 deleted the mergify/bp/main/pr-271 branch December 12, 2022 11:49
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