Closed (fixed)
Project:
Project Issue File Review
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Feb 2012 at 02:58 UTC
Updated:
10 Mar 2012 at 22:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
rfayWouldn't a preg_match() do this more elegantly?
Comment #2
jthorson commentedWell ... I said 'suggested approach', not 'best suggested approach'. Two days of line-by-line debugging, and now 'line by line' is the first thing to pop into my head. ;)
Comment #3
jthorson commentedUntested, but should be close. Need to do some research and find some patches to test it.
Comment #4
jthorson commentedTested on scratchtestbot and commited to 6.x-2.x (a0fe4fb).
Comment #5
jthorson commentedComment #6
jthorson commentedOoops ... this broke the 'non-unix-line-endings' confirmation patch.
Comment #7
jthorson commentedCommitted to 6.x-2.x (commit 000daa5).