Opened 7 years ago
Last modified 7 years ago
#13183 assigned task
Add copy constructor for BStringRef
| Reported by: | owenca | Owned by: | nobody |
|---|---|---|---|
| Priority: | normal | Milestone: | Unscheduled |
| Component: | Kits/Support Kit | Version: | R1/Development |
| Keywords: | BStringRef | Cc: | |
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
http://pulkomandy.github.io/ #32: Add a copy constructor to BStringRef
Attachments (2)
Change History (5)
by , 7 years ago
| Attachment: | 0001-added-copy-constructor-for-BStringRef-PVS-issue-32.patch added |
|---|
comment:1 by , 7 years ago
| patch: | 0 → 1 |
|---|
by , 7 years ago
| Attachment: | BStringRef.png added |
|---|
comment:2 by , 7 years ago
I have deprecated this API because it is not needed, however, the class was kept for binary compatibility. I think it should not be allowed to keep a BStringRef, so I would make the copy constructor private and consider removing the operator= if possible (it is usually not used, since the reference is often converted to a char or to a pointer to the string internal data?).
comment:3 by , 7 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
Note:
See TracTickets
for help on using tickets.
testing