Modify

Opened 6 years ago

Closed 6 years ago

#6905 closed Bugs (fixed)

bcp --namespace misses a boost namespace in emplace_args.hpp

Reported by: lukester_null@… Owned by: Daniel James
Milestone: To Be Determined Component: unordered
Version: Boost 1.49.0 Severity: Problem
Keywords: Cc:

Description

Using bcp to create a custom namespace doesn't convert:

BOOST_UNORDERED_CONSTRUCT_FROM_TUPLE(10, boost)

Attaching a possible fix (only tried on VC++ 10 so far).

Regards

Luke Elliott.

Attachments (2)

emplace_args.hpp (19.4 KB) - added by lukester_null@… 6 years ago.
Possible fix.
emplace_args.diff (3.3 KB) - added by lukester_null@… 6 years ago.
Possible fix as diff.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by lukester_null@…

Attachment: emplace_args.hpp added

Possible fix.

comment:1 Changed 6 years ago by Daniel James

Status: newassigned

Can you post it as a diff please?

I had a quick look and I think you're illegally concatenating :: and tuple. A preprocessor concatenation has to create a single token but ::tuple is two tokens. Visual C++ accepts things like that, but it shouldn't really. You probably don't need a concatenation at all, could just say namespace_ tuple and that should work.

I'll look into this in more detail later.

Changed 6 years ago by lukester_null@…

Attachment: emplace_args.diff added

Possible fix as diff.

comment:2 Changed 6 years ago by lukester_null@…

Sure - attached without ##.

comment:3 Changed 6 years ago by Daniel James

(In [78491]) Unordered: Fix namespaces for renaming in bcp. Refs #6905.

comment:4 Changed 6 years ago by Daniel James

Resolution: fixed
Status: assignedclosed

(In [78534]) Unordered: Fix bcp namespace fix. Fixes #6905.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Daniel James.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.