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"

Bug? IsA(SomeBase) returns true if SomeBase is NULL

The function bool UObjectBaseUtility::IsA( const UClass* SomeBase ) const returns true if SomeBase is NULL. Shouldn't it return false instead? It makes more sense to me.

Product Version: Not Selected
Tags:
more ▼

asked Apr 23 '14 at 05:34 PM in Bug Reports

avatar image

Ixiguis
1.2k 58 61 253

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

1 answer: sort voted first

Yes its a bug and i seen somebody submitted fix for that in github

more ▼

answered Apr 23 '14 at 05:37 PM

avatar image

Shadowriver
37.1k 935 172 1116

avatar image Ixiguis Apr 23 '14 at 05:41 PM

It was me who posted there, but I've closed the pull request because it was including other commits there. So this still needs Epic's attention, if it is indeed a bug.

https://github.com/EpicGames/UnrealEngine/pull/69

avatar image Shadowriver Apr 23 '14 at 05:47 PM

Well object funtion called "is a" not null, so logicly it is a bug imo. Dont be shy and correct your pull request as suggested, epic will jurge it if should be fixed there.

avatar image Ixiguis Apr 23 '14 at 05:53 PM

To be honest, it feels like too much work (creating a new fork, redownloading the engine, etc) for such a small change. I am not very experienced with GitHub.

avatar image Shadowriver Apr 23 '14 at 06:02 PM

Well you can edit in github:) forking is not that slow, open file in your fork and and there should be edit button, edit automatically will create commit and then pull request

avatar image Ixiguis Apr 23 '14 at 06:16 PM

Ohhh cool, thanks for the tip! I was thinking I'd have to fork and download the entire engine again, in a separate folder on my disk.

New pull request: https://github.com/EpicGames/UnrealEngine/pull/73

(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