Import 48fc9a1c69: Catch null entity pointer in ETR; removed minor cruft (by Caitlyn)

[daleglass]

This is an automated proposal to look at a commit made by Caitlyn and import it into Overte

Commit:	48fc9a1c69cacc2ab6b2bb5e05fc096fda8947b5
Author:	Caitlyn
Date:	Mon, 06 Apr 2020 13:05

Catch null entity pointer in ETR; removed minor cruft 

Stats:

Filename	Stats	Lines	Added	Removed	Lines in blame
libraries/entities-renderer/src/EntityTreeRenderer.cpp		47	19	28	1
libraries/entities/src/EntityItem.cpp		5	5	0	5
libraries/entities/src/EntityItem.h		14	5	9	5
3 files	-	66	29	37	11

To work on this, please assign the issue to yourself, then look at the commit and decide whether this would be a good addition to Overte.

• If the commit is useful, tag it with "Tivoli: Keep", and keep it open until it's merged.
• If the commit is not useful, tag it with "Tivoli: Discard", and close it.
• If the commit is not useful right now, but might be later, tag it with "Tivoli: Maybe later", and close it.
• If it's hard to decide, tag it with "Tivoli: Discuss", and keep it open.

Useful commits should be submitted as a PR against Overte. Tag this issue in PR, so that it's automatically closed once the PR is merged.

You can cherry-pick this issue with this command:

git cherry-pick 48fc9a1c69cacc2ab6b2bb5e05fc096fda8947b5

[daleglass]

static entities