Hi Lance,

The logic looks correct but enquoteLiteral is going to create an lot of garbage Strings, about one for every character. I'm not sure Streams are the right tool for this task.

Can't String.replace("'", "''") do the same job?

Roger


On 10/20/2015 4:19 PM, Lance Andersen wrote:
Hi,

Looking for a reviewer for 
http://cr.openjdk.java.net/~lancea/8139056/webrev.00/.

This change Adds two convenience methods to Statement.java:
- enquoteLiteral which returns a string enclosed in single quotes where all occurrences of a single quote are converted to two single quotes - enquoteIdentifier returns a SQL identifier


The CCC has been approved.

Best
Lance




Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
[email protected]




Reply via email to