[ic] Redundant code in DBI.pm?

[Peter]

On 03/14/2015 04:23 AM, Mike Heins wrote:
> If anyone wonders why I bothered with any of this, at one point I
> was working on a database configuration system for Minivend 4. Spaces
> in field names were not on my radar (even if they were supported in
> SQL, which I don't think they were at that time), so it seemed like an
> easy way to specify a field name and definition as in SQL. This never
> was really used, as Akopia took over custody and we went in a different
> direction.

So that's the other thing to consider.  If we rip this out entirely we
gain support for spaces in identifiers (especially with the addition of
QUOTE_IDENTIFIERS).  As it is now, space in an identifier will still
bork with that section of code.

We might possibly keep support for both with a lookbehind assertion that
checks for the closing quote (that matches the opening quote) followed
by space instead of just space.


Peter