fsfw/fsfw
11
3
Fork 6
Code Issues 32 Pull Requests 4 Releases 10 Wiki Activity

StorageManagerIF: improved naming, new getter function #354

Closed
muellerr wants to merge 7 commits from KSat:mueller/storage-man-update into development
pull from: KSat:mueller/storage-man-update
merge into: fsfw:development
Conversation 0 Commits 7 Files Changed -
muellerr commented 2021-01-18 20:06:48 +01:00
Owner
Copy Link

I think the previous naming was a bit confusing. The number of pools is now exposed by a getter function instead of exposing a member variable directly.

Maybe it would also be nice to expose more information about the pools like the number of elements in a pool and the bucket sizes through getter functions..

I think the previous naming was a bit confusing. The number of pools is now exposed by a getter function instead of exposing a member variable directly. Maybe it would also be nice to expose more information about the pools like the number of elements in a pool and the bucket sizes through getter functions..
muellerr added the
feature
API Change
 labels 2021-01-18 20:06:48 +01:00
muellerr added 1 commit 2021-01-18 20:06:49 +01:00
improved naming ,new getter function d4e392389a
muellerr changed title from improved naming ,new getter function to StorageManagerIF: improved naming, new getter function 2021-01-18 20:08:13 +01:00
muellerr added 1 commit 2021-01-18 20:21:37 +01:00
doc update e1423c66c7
gaisser added 1 commit 2021-01-19 14:58:51 +01:00
Merge branch 'development' into mueller/storage-man-update fab97ed981
muellerr added 1 commit 2021-01-19 15:08:20 +01:00
better naming b571e93da8
gaisser added 1 commit 2021-01-19 15:39:13 +01:00
Merge branch 'development' into mueller/storage-man-update 3c91c36970
muellerr added 35 commits 2021-01-26 14:26:39 +01:00
Merge remote-tracking branch 'origin/mueller/test-update' into mueller/bundled-update c5212226b8
Merge branch 'mueller/test-update' into mueller/bundled-update 5b9c0e013e
added service 3 cmake support 219c057fb9
added srv3 8ef6283bf4
made warning flag optional 1ac2479b28
Merge remote-tracking branch 'upstream/mueller/master' into mueller/master bd5cc7ae3e
Merge remote-tracking branch 'upstream/mueller/master' into mueller/master f05af8f54a
resolved merge conflicts a5e20657c6
Merge remote-tracking branch 'upstream/mueller/master' into mueller/master 58c5b02113
Merge remote-tracking branch 'upstream/mueller/master' into mueller/master d8423be98c
Merge remote-tracking branch 'upstream/mueller/master' into development f6c88886b8
moved read helper into correct folder d6b0af3be7
Merge remote-tracking branch 'upstream/mueller/master' into development 4cf15e9ec6
Merge remote-tracking branch 'upstream/development' into eive/develop 65f493b230
cmake lists update ffb6208b24
Merge branch 'development' into eive/shadow-local-warning 97e86b19f0
Merge remote-tracking branch 'upstream/development' into eive/shadow-local-warning 387bb0c434
Merge branch 'eive/shadow-local-warning' of https://egit.irs.uni-stuttgart.de/eive/fsfw into eive/shadow-local-warning 0d9bb5b35e
Merge pull request 'Warning on local shadowing can be disabled optionally for GCC' (#357) from eive/fsfw:eive/shadow-local-warning into development 2f64c16a8a 
Reviewed-on: #357
improved options b23309a5f4
Merge branch 'eive/shadow-local-warning' into eive/develop 55263fe4d8
Merge branch 'development' into eive/shadow-local-warning fd5252bb57
updating cmakelists 2e9a855181
Merge branch 'eive/shadow-local-warning' of https://egit.irs.uni-stuttgart.de/eive/fsfw into eive/shadow-local-warning 94d1b5acae
Merge remote-tracking branch 'upstream/development' into eive/develop 8f83f7becd
added missing endif e0f8113eac
Merge branch 'eive/develop' into eive/shadow-local-warning 6a5f5af233
small fix 50733e223a
important bugfix 8850b770ee
Merge branch 'development' into mueller/hk-bugfix a85c94ea80
Merge pull request 'important bugfix' (#360) from KSat/fsfw:mueller/hk-bugfix into development f11e7407d6 
Reviewed-on: #360
Merge branch 'development' into eive/shadow-local-warning d39c729a16
Merge pull request 'improved options' (#359) from eive/fsfw:eive/shadow-local-warning into development 7310d2b5fe 
Reviewed-on: #359
Merge remote-tracking branch 'upstream/development' into mueller/storage-man-update caed8be21d
Merge remote-tracking branch 'upstream/development' into mueller/storage-man-update b578fb93b5
muellerr closed this pull request 2021-01-26 14:26:57 +01:00
muellerr deleted branch mueller/storage-man-update 2021-01-26 14:27:02 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
API Change
Changes the API, so Users need to adapted
Breaking API Change
Changes the API in a non backwards compatible way
bug
Something is not working
build
Issues related to the buildsystem, including CI
cosmetics
Small change most users won't notice
Documentation
Improvement of code documentation
duplicate
This issue or pull request already exists
feature
New feature
help wanted
Need some help
hotfix
Quick or important fix that is merged directly into master, causing a new revision
invalid
#worksforme
question
More information is needed
Refactor
Contains a refactoring of some code
Tests
Issues related to testing code
wontfix
This won't be fixed
No Label
API Change
feature
Milestone
No items
No Milestone
Assignees
Clear assignees
ahinkel dwoodward gaisser hanke jenkins_buildfix lrajer meierj mohr muellerr taheran tbaumgartl
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: fsfw/fsfw#354
No description provided.
Delete Branch "KSat:mueller/storage-man-update"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?