job000021 #7

# 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:	job000021

Status:	closed

Project:	jam

Severity:	B

ReportedBy:	[email protected]

ReportedDate:	2002/04/01 14:48:16

ModifiedBy:	shawn_hladky

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

Description:
	input:
	    x = [ MATCH (foo)(.*) : foo ] ;
	    ECHO -$(x)+ ;
	
	output:
	    -foo+
	
	Shouldn't this print:
	
	    -foo+ -+

DevNotes:
	(rmg, 4/1/2002)
	
	It appears that "trailing" empty-string matches are
	not being recognized by the code in builtins.c:
	
	The following change gets the expected behavior:
	
	  263c263
	  <                   if( re->startp[top] != re->endp[top] )
	  ---
	  >                   if( re->startp[top] != NULL )
	
	but I'm Checking with Christopher to make sure I haven't missed
	his intent, before submitting the fix.