Open Bug 1731077 Opened 3 years ago Updated 3 years ago

Mark std::tuple as mem-movable?

Categories

(Developer Infrastructure :: Source Code Analysis, enhancement, P3)

Product:
Developer Infrastructure
Component:
Source Code Analysis
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: saschanaz, Unassigned)

References

Details

https://searchfox.org/mozilla-central/rev/4d90ff4537330d6b17cb956c0fadf759086d9bb7/build/clang-plugin/MemMoveAnnotation.h#40

isNameExcepted() lists std::pair as mem-movable but not std::tuple. Should it be listed together?

It seems to me that std::pair<T, U> should only be mem-movable if T and U are -- does the plugin check that?

Doesn't seems so to me, line 26 already implies the detection is not perfect.

Severity: -- → S3
Type: task → enhancement
Priority: -- → P3
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.