Re: Fixing string offsets of strings.

From: Date: Mon, 05 Dec 2011 14:59:50 +0000
Subject: Re: Fixing string offsets of strings.
References: 1 2  Groups: php.internals 
Request: Send a blank email to internals+get-56779@lists.php.net to get a copy of this message
Hi Alan: > > https://bugs.php.net/patch-display.php?bug=60362&patch=fix_disabling_bad_string_offsets&revision=1323002696 Thanks so much for the patch. Looking at the source I noticed some small things (care to build a bike shed with me? :). Many of the test EXPECT's have %d changed to hard coded line numbers and others have %s changed to hard coded file names. In some of them, that's all that changed. I assume that will be cleaned up before committing. There are some extraneous whitespace changes in the source. I tried to apply the patch to 54 and trunk via "patch < bug.diff" (which usually works fine) and all hunks failled. Not sure what I'm doing wrong. Thanks, --Dan -- T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y data intensive web and database programming http://www.AnalysisAndSolutions.com/ 4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409

« previous php.internals (#56779) next »