Actions
Bug #5809
closedsmb: convert transaction list to vecdeque (6.0.x backport)
Added by Shivani Bhardwaj over 1 year ago. Updated 11 months ago.
Affected Versions:
Effort:
Difficulty:
Label:
Updated by Philippe Antoine over 1 year 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 over 1 year ago
- Target version changed from 6.0.10 to 6.0.11
Updated by Philippe Antoine over 1 year ago
So, do you think we should backport all protocols ? (I would reject this for 6)
Updated by Victor Julien over 1 year 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 about 1 year ago
So @Victor Julien , should this be done for 6.0.11 ?
Updated by Philippe Antoine about 1 year ago
- Target version changed from 6.0.11 to 6.0.12
Updated by Victor Julien about 1 year ago
- Target version changed from 6.0.12 to 6.0.13
Updated by Philippe Antoine 11 months ago
- Status changed from Assigned to In Review
Updated by Philippe Antoine 11 months ago
- Status changed from In Review to Closed
Actions