Changes between Initial Version and Version 1 of Ticket #163, comment 10


Ignore:
Timestamp:
05/06/13 13:21:29 (8 years ago)
Author:
David Sommerseth
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #163, comment 10

    initial v1  
    11Thank you very much for your thorough review of this issue, Waffelman.  Now it is fully possible to understand why pfs is NULL, which definitely was the most important thing here.
    22
    3 That it is NULL due to no parsed pf file, makes a lot of sense.  So I agree, the patch can now be included.  The other approach and writing to this file, to give it some valid contents seems like the wrong solution to me.
     3That it is NULL due to no parsed pf contents (file or management interface), makes a lot of sense.  So I agree, the patch can now be included.  The other approach and writing to this file, to give it some valid contents seems like the wrong solution to me.
    44
    5 And I checked quickly the call paths for pf_cn_test(), which always called via pf_c2c_test().  So it seems to be something like this:
     5And I checked quickly the call paths for pf_cn_test(), which always is called via pf_c2c_test().  So it seems to be something like this:
    66
    77{{{