job000020 #7

  • //
  • spec/
  • job/
  • job000020
  • View
  • Commits
  • Open Download .zip Download (2 KB)
# Perforce Public Depot Jobs
#
#  Job:           The job name. 'new' generates a sequenced job number.
#  Status:        Job status; [open/closed/suspended].  Required
#  Project:       The project this job is for [none/cdsp4/jam/p4hl]  Optional.
#  Severity:      [A/B/C] (A is highest)  Required.
#  ReportedBy     The user who created the job. Can be changed.
#  ReportedDate:  The date the job was created.  Automatic.
#  ModifiedBy:    The user who last modified this job. Automatic.
#  ModifiedDate:  The date this job was last modified. Automatic.
#  OwnedBy:       The owner, responsible for doing the job. Optional.
#  Description:   Description of the job.  Required.
#  DevNotes:      Developer's comments.  Optional.

Job:	job000020

Status:	closed

Project:	p4hl

Severity:	A

ReportedBy:	sam_stafford

ReportedDate:	2002/03/24 10:45:33

ModifiedBy:	shawn_hladky

ModifiedDate:	2008/04/13 13:00:54

OwnedBy:	""

Description:
	(P4HL support code)  Attempts to store revs even if they can't be indexed by change.  This
	creates extra bogus FileRevs with a change of "" - this only applies to changes submitted
	by ancient servers (before submitted changes were numbered).  In P4HL this is pretty 
	harmless, but it does lead to creation of extra entities.  In other applications this could
	lead to crashes, very hard-to-track down bugs, and me being very upset at myself when I find
	out what was causing the problem.

DevNotes:
	The fix should be to reject revs if change.Length() or change.Atoi() == 0.  This should go in
	FileHead::addRev().
# Change User Description Committed
#7 default
#6 default
#5 default
#4 default
#3 default
#2 default
#1 default