x

Search in
Sort by:

Question Status:

Search help

  • Simple searches use one or more words. Separate the words with spaces (cat dog) to search cat,dog or both. Separate the words with plus signs (cat +dog) to search for items that may contain cat but must contain dog.
  • You can further refine your search on the search results page, where you can search by keywords, author, topic. These can be combined with each other. Examples
    • cat dog --matches anything with cat,dog or both
    • cat +dog --searches for cat +dog where dog is a mandatory term
    • cat -dog -- searches for cat excluding any result containing dog
    • [cats] —will restrict your search to results with topic named "cats"
    • [cats] [dogs] —will restrict your search to results with both topics, "cats", and "dogs"

ambiguous symbol in PointerIsConvertibleFromTo.h

Other libraries like to have their own typedefs for things like uint8, etc and when they are included you get errors in lines 12 and 13 in PointerIsConvertibleFromTo.h due to ambiguous symbols.

Would it be possible to update lines 12 and 13 of engine\source\runtime\core\public\templates\PointerIsConvertibleFromTo.h in mainline

From

     static uint8  Test(...);
     static uint16 Test(To*);

To

     static FGenericPlatformTypes::uint8  Test(...);
     static FGenericPlatformTypes::uint16 Test(To*);

to prevent this ambiguity?

Product Version: UE 4.11
Tags:
more ▼

asked Apr 12 '16 at 08:22 PM in Bug Reports

avatar image

GeorgeDavis
1 1 2

(comments are locked)
10|2000 characters needed characters left
Viewable by all users

1 answer: sort voted first

Hi GeorgeDavis,

At first glance, it doesn't appear that there would be a problem with this. Normally when it comes to suggestions like this, we suggest that you submit the change on GitHub as a pull request. We can then assign someone to review your suggestion.

If you can't submit a pull request, or prefer not to, let me know and I will enter a feature request for this.

Tim

more ▼

answered Apr 15 '16 at 06:51 PM

avatar image Tim C ♦♦ STAFF Apr 29 '16 at 03:11 PM

Hi GeorgeDavis,

We have not heard back from you for some time. If you are not able to submit this change as a pull request, please let us know. For now, I will be marking this question as resolved, but please feel free to add a comment and re-open it if necessary.

Tim

(comments are locked)
10|2000 characters needed characters left
Viewable by all users
Your answer
toggle preview:

Up to 5 attachments (including images) can be used with a maximum of 5.2 MB each and 5.2 MB total.

Follow this question

Once you sign in you will be able to subscribe for any updates here

Answers to this question