[modernize] use-default-member-init

[Rechi]

Description

run clang-tidy modernize-use-default-member-init check and applied the suggested fixes
run clang-tidy-diff with modernize-use-nullptr check and applied the suggested fixes
run clang-tidy modernize-use-default-member-init check and applied the suggested fixes
run clang-tidy-diff with modernize-use-equals-default check and applied the suggested fixes

Types of change

• Bug fix (non-breaking change which fixes an issue)
• Clean up (non-breaking change which removes non-working, unmaintained functionality)
• Improvement (non-breaking change which improves existing functionality)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that will cause existing functionality to change)
• Cosmetic change (non-breaking change that doesn't touch code)
• None of the above (please explain below)

Checklist:

• My code follows the Code Guidelines of this project
• My change requires a change to the documentation, either Doxygen or wiki
• I have updated the documentation accordingly
• I have read the Contributing document
• I have added tests to cover my change
• All new and existing tests passed

[ksooo]

Shouldn't we also init m_gifFile with a nullptr here (line 113)?

[Rechi]

Have a look at the GifHelper constructor.

xbmc/tools/depends/native/TexturePacker/src/decoder/GifHelper.cpp

Lines 46 to 56 in dd178e7

	GifHelper::GifHelper() :
	m_imageSize(0),
	m_pitch(0),
	m_loops(0),
	m_numFrames(0),
	m_filename(""),
	m_gif(nullptr),
	m_pTemplate(nullptr)
	{
	m_gifFile = new CFile();
	}

[ksooo]

lgtm, thank you