diff options
author | drh <drh@noemail.net> | 2018-04-16 10:34:13 +0000 |
---|---|---|
committer | drh <drh@noemail.net> | 2018-04-16 10:34:13 +0000 |
commit | 25c3b8caf4f19658cb34ad33686fcf9f30873907 (patch) | |
tree | 4ec0a4f1f5ce0f7228ad85d15d1c122f1b66b002 /src/resolve.c | |
parent | 54f2cd905b2d81e5bbce99d106f95c343b12579b (diff) | |
download | sqlite-25c3b8caf4f19658cb34ad33686fcf9f30873907.tar.gz sqlite-25c3b8caf4f19658cb34ad33686fcf9f30873907.zip |
Reduce the size of the NameContext object by grouping seldom-used fields
into a union.
FossilOrigin-Name: dba3095feeeb55b5c0ebe33bdd4be8ba1f24478406685d3a042a69d9c380e742
Diffstat (limited to 'src/resolve.c')
-rw-r--r-- | src/resolve.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/src/resolve.c b/src/resolve.c index 7ae49bde5..c8059df97 100644 --- a/src/resolve.c +++ b/src/resolve.c @@ -383,10 +383,12 @@ static int lookupName( ** is supported for backwards compatibility only. Hence, we issue a warning ** on sqlite3_log() whenever the capability is used. */ - if( (pEList = pNC->pEList)!=0 - && zTab==0 + if( (pNC->ncFlags & NC_UEList)!=0 && cnt==0 + && zTab==0 ){ + pEList = pNC->uNC.pEList; + assert( pEList!=0 ); for(j=0; j<pEList->nExpr; j++){ char *zAs = pEList->a[j].zName; if( zAs!=0 && sqlite3StrICmp(zAs, zCol)==0 ){ @@ -915,8 +917,8 @@ static int resolveOrderByTermToExprList( memset(&nc, 0, sizeof(nc)); nc.pParse = pParse; nc.pSrcList = pSelect->pSrc; - nc.pEList = pEList; - nc.ncFlags = NC_AllowAgg; + nc.uNC.pEList = pEList; + nc.ncFlags = NC_AllowAgg|NC_UEList; nc.nErr = 0; db = pParse->db; savedSuppErr = db->suppressErr; @@ -1299,7 +1301,9 @@ static int resolveSelectStep(Walker *pWalker, Select *p){ ** Minor point: If this is the case, then the expression will be ** re-evaluated for each reference to it. */ - sNC.pEList = p->pEList; + assert( (sNC.ncFlags & (NC_UAggInfo))==0 ); + sNC.uNC.pEList = p->pEList; + sNC.ncFlags |= NC_UEList; if( sqlite3ResolveExprNames(&sNC, p->pHaving) ) return WRC_Abort; if( sqlite3ResolveExprNames(&sNC, p->pWhere) ) return WRC_Abort; |