This is an archive of the discontinued LLVM Phabricator instance.

Remove definition of ~PlatformNetBSD(), since its declaration is marked as '= default'
ClosedPublic

Authored by krytarowski on Oct 13 2015, 1:41 PM.

Reviewers

joerg
brucem

Commits

Summary

Local definition of ~PlatformNetBSD() results with a compiler error.

krytarowski retitled this revision from to Remove definition of ~PlatformNetBSD(), since its declaration is marked as '= default'.

krytarowski updated this object.

krytarowski added a reviewer: joerg.

krytarowski set the repository for this revision to rL LLVM.

lgtm.

will land shortly for you.

This revision is now accepted and ready to land.Oct 13 2015, 4:20 PM

This revision was automatically updated to reflect the committed changes.

Thank you!

Path

Size

lldb/

trunk/

source/

Plugins/

Platform/

NetBSD/

13 lines

	Show First 20 Lines • Show All 284 Lines • ▼ Show 20 Lines
	/// Default Constructor			/// Default Constructor
	//------------------------------------------------------------------			//------------------------------------------------------------------
	PlatformNetBSD::PlatformNetBSD (bool is_host) :			PlatformNetBSD::PlatformNetBSD (bool is_host) :
	Platform(is_host),			Platform(is_host),
	m_remote_platform_sp()			m_remote_platform_sp()
	{			{
	}			}

	//------------------------------------------------------------------
	/// Destructor.
	///
	/// The destructor is virtual since this class is designed to be
	/// inherited from by the plug-in instance.
	//------------------------------------------------------------------
	PlatformNetBSD::~PlatformNetBSD()
	{
	}

	//TODO:VK: inherit PlatformPOSIX


	bool			bool
	PlatformNetBSD::GetRemoteOSVersion ()			PlatformNetBSD::GetRemoteOSVersion ()
	{			{
	if (m_remote_platform_sp)			if (m_remote_platform_sp)
	return m_remote_platform_sp->GetOSVersion (m_major_os_version,			return m_remote_platform_sp->GetOSVersion (m_major_os_version,
	m_minor_os_version,			m_minor_os_version,
	m_update_os_version);			m_update_os_version);
	return false;			return false;
	▲ Show 20 Lines • Show All 383 Lines • Show Last 20 Lines