Skip site navigation (1) Skip section navigation (2)

Re: code coverage patch

From: Korry Douglas <korry(dot)douglas(at)enterprisedb(dot)com>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org, Michelle Caisse <Michelle(dot)Caisse(at)sun(dot)com>
Subject: Re: code coverage patch
Date: 2008-08-28 13:08:46
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers
> The problem with those files is that the source file contains lines  
> like this:
> #line 1042 ""
> but that source file does not exist, as it is renamed to gram.c.
> We could fix that in one of two ways:
> 1) Use bison's -o option to put the output file in the right place  
> directly,
> if we are dealing with bison (and don't bother to support code  
> coverage
> analysis with other yaccs), or
> 2) Run a pattern replacement across the grammar output files as  
> their are
> renamed.

Why not use the %output directive in the grammar file instead; that  
way you don't need to add any special flags to the Makefile.

			-- Korry

In response to


pgsql-hackers by date

Next:From: Peter EisentrautDate: 2008-08-28 13:32:45
Subject: Re: code coverage patch
Previous:From: Gregory StarkDate: 2008-08-28 11:56:56
Subject: Re: code coverage patch

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group