trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From DaveBirdsall <...@git.apache.org>
Subject [GitHub] trafodion pull request #1405: [TRAFODION-2873] Removed usage of system new f...
Date Thu, 18 Jan 2018 22:46:03 GMT
Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1405#discussion_r162492113
  
    --- Diff: core/sql/cli/Cli.cpp ---
    @@ -9611,12 +9611,28 @@ Lng32 SQLCLI_LOBddlInterface
     	    
     	  } // for
     
    +        //Initialize LOB interface 
    +        
    +        Int32 rc= ExpLOBoper::initLOBglobal(exLobGlob,currContext.exHeap(),&currContext,hdfsServer,hdfsPort);
    +        if (rc)
    +          {
    +            {
    --- End diff --
    
    It doesn't hurt anything, but I'm curious: Why two sets of braces? Looks like one would
do?


---

Mime
View raw message