Opened 6 years ago
Closed 4 years ago
#7791 closed defect (fixed)
Unprefixed defines
| Reported by: | Gramner | Owned by: | |
|---|---|---|---|
| Priority: | normal | Component: | avutil |
| Version: | git-master | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Reproduced by developer: | no | |
| Analyzed by developer: | no |
Description
The assembly support macros defined in the libavutil/mem.h public header does not have any prefixes which may cause namespace collisions when included by users.
They should either have some appropriate prefix added, or alternatively moved to some internal header since having them public seems to be of questionable usefulness.
This may apply to more defines in other headers as well which perhaps might be worth investigating, but DECLARE_ALIGNED happens to actually cause a name collision when libavformat/avformat.h is included by x264.
Note:
See TracTickets
for help on using tickets.
Said macros have been deprecated in e15371061d23457554d241a80dc471515ac13ad4 and will be removed in the next+1 major version (i.e. libavutil version 58). So I am closing this despite said macros still being there, as one of your proposed solutions has been implemented (or at least scheduled).