Modify

Ticket #6406 (closed Bugs: fixed)

Opened 2 years ago

Last modified 21 months ago

Use attribute names with underscores for GCC

Reported by: Petr Machata <pmachata@…> Owned by: eric_niebler
Milestone: Boost 1.51.0 Component: xpressive
Version: Boost 1.50.0 Severity: Problem
Keywords: Cc:

Description

... which should prevent accidental interactions between user-defined macros and boost internals.

Attachments

boost-xpressive.patch Download (657 bytes) - added by Petr Machata <pmachata@…> 2 years ago.

Change History

Changed 2 years ago by Petr Machata <pmachata@…>

comment:1 Changed 2 years ago by Petr Machata <pmachata@…>

  • Owner set to eric_niebler
  • Component changed from None to xpressive

comment:2 Changed 2 years ago by eric_niebler

Can you point to official gcc documentation that show that this is a supported and equivalent syntax? All I see is here:  http://gcc.gnu.org/onlinedocs/gcc/Attribute-Syntax.html, which seems to indicate that the original code is the correct form. And if the form you suggested is allowed, then is it only a recent addition?

comment:3 Changed 2 years ago by eric_niebler

  • Status changed from new to closed
  • Resolution set to invalid

Resolving as invalid for now. If you can show that this has been supported syntax for a long time, feel free to reopen.

comment:4 Changed 2 years ago by Petr Machata <pmachata@…>

I updated the ticket #6416 with some evidence that this is a long-supported feature.

comment:5 Changed 2 years ago by Petr Machata <pmachata@…>

  • Status changed from closed to reopened
  • Resolution invalid deleted

comment:6 Changed 2 years ago by Petr Machata <pmachata@…>

The double underscore feature is described here:  http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html and it has worked "forever" (I tried GCC 3.3, but reportedly it's been supported all the way back to GCC 2.95.7).

comment:7 Changed 2 years ago by eric_niebler

  • Status changed from reopened to closed
  • Resolution set to fixed

(In [76766]) use underscores in gcc attribute names, fixes #6406

comment:8 Changed 21 months ago by denis.arnaud

  • Status changed from closed to reopened
  • Version changed from Boost 1.48.0 to Boost 1.50.0
  • Resolution fixed deleted
  • Milestone changed from To Be Determined to Boost 1.51.0

The bug has been fixed on trunk ( http://svn.boost.org/svn/boost/trunk/boost/xpressive/detail/core/adaptor.hpp), but not on the release branch ( http://svn.boost.org/svn/boost/branches/release/boost/xpressive/detail/core/adaptor.hpp)...

Could you integrate the fix in the release branch as well?

comment:9 Changed 21 months ago by eric_niebler

  • Status changed from reopened to closed
  • Resolution set to fixed

Oops. Merged to release in [79614]. Thanks for spotting that.

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.