Skip to content
Snippets Groups Projects
  • Bruce Momjian's avatar
    e30b283f
    Attached is my attempt to clean up the horrors of the ExecSQL() method in · e30b283f
    Bruce Momjian authored
    the JDBC driver.
    
    I've done this by extracting it into a new method object called
    QueryExecutor (should go into org/postgresql/core/) and then taking it
    apart into different methods in that class.
    
    A short summary:
    
    * Extracted ExecSQL() from Connection into a method object called
      QueryExecutor.
    
    * Moved ReceiveFields() from Connection to QueryExecutor.
    
    * Extracted parts of the original ExecSQL() method body into smaller
      methods on QueryExecutor.
    
    * Bug fix: The instance variable "pid" in Connection was used in two
      places with different meaning. Both were probably in dead code, but it's
      fixed anyway.
    
    Anders Bengtsson
    e30b283f
    History
    Attached is my attempt to clean up the horrors of the ExecSQL() method in
    Bruce Momjian authored
    the JDBC driver.
    
    I've done this by extracting it into a new method object called
    QueryExecutor (should go into org/postgresql/core/) and then taking it
    apart into different methods in that class.
    
    A short summary:
    
    * Extracted ExecSQL() from Connection into a method object called
      QueryExecutor.
    
    * Moved ReceiveFields() from Connection to QueryExecutor.
    
    * Extracted parts of the original ExecSQL() method body into smaller
      methods on QueryExecutor.
    
    * Bug fix: The instance variable "pid" in Connection was used in two
      places with different meaning. Both were probably in dead code, but it's
      fixed anyway.
    
    Anders Bengtsson