[SERVER-69728] Remove BufBuilder::appendChar in favour of BufBuilder::appendByte Created: 15/Sep/22 Updated: 12/Jun/23 Resolved: 24/Apr/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Jordi Olivares Provencio | Assignee: | Backlog - Storage Execution Team |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | techdebt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Storage Execution
|
||||||||
| Participants: | |||||||||
| Description |
|
The usage of BufBuilder::appendChar and the rest of the append*(char) family seems to be to append a single byte. To clarify the intent behind this we should define a method appendByte(std::byte) and replace the original method so that the meaning is clearly conveyed.
|