[Xapian-tickets] [Xapian] #239: makedepend gets confused with multiple #include "filename..." in same program

Xapian nobody at xapian.org
Wed Mar 10 00:23:49 GMT 2010


#239: makedepend gets confused with multiple #include "filename..." in same
program
-------------------------------+--------------------------------------------
 Reporter:  caa                |        Owner:  caa      
     Type:  defect             |       Status:  reopened 
 Priority:  normal             |    Milestone:  1.0.19   
Component:  MSVC makefiles     |      Version:  SVN trunk
 Severity:  major              |   Resolution:           
 Keywords:                     |    Blockedby:           
 Platform:  Microsoft Windows  |     Blocking:           
-------------------------------+--------------------------------------------
Changes (by olly):

 * cc: olly (removed)
  * milestone:  => 1.0.19


Comment:

 In normal make it should make no difference whether the prerequisites are
 one per rule or many per rule, but nmake does sometimes seem to be oddly
 different.  And splitting them should avoid any issues with limited line
 length.

 Applied the patch with a couple of tweaks in r14148.

 I avoided appending a string to objfile which might overflow it, and
 instead just write it to the output each time in the loop below.

 GCC warns about a construct in your copying loop - I think it is undefined
 behaviour because ch on the LHS could be evaluated before, after, or
 concurrently with ch++ on the right (isn't C lovely?):

 {{{
 #!c
 objfile[ch] = buf[ch++];
 }}}

 So I've tweaked that to be a for loop with the increment separate.

 I'm reluctant to commit unused files as they'll bloat every checkout a
 bit, and sooner or later will end up out of sync.  Let's just decree the
 nmake file as the supported way to build.  If anyone really wants a
 project file they can create one easily enough.

 So unless I broke something, this is ready for backporting.

-- 
Ticket URL: <http://trac.xapian.org/ticket/239#comment:17>
Xapian <http://xapian.org/>
Xapian



More information about the Xapian-tickets mailing list