[SERVER-29775] DataView and BufReader should either default to LittleEndian or require explicit endianness Created: 21/Jun/17 Updated: 06/Dec/22 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | Internal Code |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Mathias Stearn | Assignee: | Backlog - Service Architecture |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | gm-ack, move-sa | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Service Arch
|
||||||||
| Sprint: | Service Arch 2022-05-02 | ||||||||
| Participants: | |||||||||
| Description |
|
It is way too easy to forget the annotations, and they are incompatible with some forms of the methods. Since LittleEndian is almost always what we want and NativeEndian is almost never, it seems like LittleEndian is the better implict default. Either way, I think we should never implicitly default to Native and require making it explicit. |
| Comments |
| Comment by Matt Diener (Inactive) [ 02/May/22 ] |
|
This seems like it will help avoid mistakes. Unclear how hard it will be. |
| Comment by Lauren Lewis (Inactive) [ 21/Dec/21 ] |
|
We haven’t heard back from you in at least 1 year, so I'm going to close this ticket. If this is still an issue for you, please provide additional information and we will reopen the ticket. |