Actions
Bug #5809
closedsmb: convert transaction list to vecdeque (6.0.x backport)
Added by Shivani Bhardwaj almost 2 years ago. Updated over 1 year ago.
Affected Versions:
Effort:
Difficulty:
Label:
Updated by Philippe Antoine almost 2 years ago
Do we really want a backport fhere ?
For instance, the DNS VecDeque has not been backported, (commit 31894147884af3e7151b4d653e5268a0b0477db8 in master)
There is only one protocol where VecDeque has been back ported : MQTT...
Updated by Victor Julien almost 2 years ago
- Target version changed from 6.0.10 to 6.0.11
Updated by Philippe Antoine almost 2 years ago
So, do you think we should backport all protocols ? (I would reject this for 6)
Updated by Victor Julien almost 2 years ago
Needs a bit more discussion. For now I just moved it out of the 6.0.10.
Wrt whether we should, I guess it comes down to cost, benefit and risk. Cost would be dev time and possible negative overall perf impact, benefit is fairly obvious and risk would be about regressions.
Updated by Philippe Antoine over 1 year ago
So @Victor Julien , should this be done for 6.0.11 ?
Updated by Philippe Antoine over 1 year ago
- Target version changed from 6.0.11 to 6.0.12
Updated by Victor Julien over 1 year ago
- Target version changed from 6.0.12 to 6.0.13
Updated by Philippe Antoine over 1 year ago
- Status changed from Assigned to In Review
Updated by Philippe Antoine over 1 year ago
- Status changed from In Review to Closed
Actions