This is an archive of the discontinued LLVM Phabricator instance.

Fix OrderField's copy constructor
ClosedPublic

Authored by thopre on Jan 15 2021, 3:54 PM.

Reviewers

Summary

OrderField's copy constructor has a typo in the constructor to call.
This commit fixes it to refer to OrderField's constructor.

Build Status

Buildable 85443
Build 116678: arc lint + arc unit

thopre requested review of this revision.Jan 15 2021, 3:54 PM

This revision is now accepted and ready to land.Jan 16 2021, 5:26 AM

Path

Size

lnt/

server/

db/

2 lines

Show First 20 Lines • Show All 286 Lines • ▼ Show 20 Lines	def __init__(self, name, ordinal):
# Column instance for fields which have been bound (non-DB		# Column instance for fields which have been bound (non-DB
# parameter). This is provided for convenience in querying.		# parameter). This is provided for convenience in querying.
self.column = None		self.column = None

def __repr__(self):		def __repr__(self):
return '%s%r' % (self.__class__.__name__, (self.name, self.ordinal))		return '%s%r' % (self.__class__.__name__, (self.name, self.ordinal))

def __copy__(self):		def __copy__(self):
return Ordinal(self.name, self.ordinal)		return OrderField(self.name, self.ordinal)


class RunField(FieldMixin, Base):		class RunField(FieldMixin, Base):
__tablename__ = 'TestSuiteRunFields'		__tablename__ = 'TestSuiteRunFields'

id = Column("ID", Integer, primary_key=True)		id = Column("ID", Integer, primary_key=True)
test_suite_id = Column("TestSuiteID", Integer, ForeignKey('TestSuite.ID'),		test_suite_id = Column("TestSuiteID", Integer, ForeignKey('TestSuite.ID'),
index=True)		index=True)
▲ Show 20 Lines • Show All 223 Lines • Show Last 20 Lines