Actions
Bug #2384
closedKeyChain regressions: impossible to create KeyChain using custom TPM/PIB
Start date:
01/15/2015
Due date:
% Done:
100%
Estimated time:
Description
ndn-cxx:commit:4154634058a33b1c1b89198af568bdefa1a88fea moved  implementation of template<class KeyChainTraits> KeyChain(KeyChainTraits traits);  which defeated the purpose of that method.
KeyChain, PIB, TPM needs some refactoring to allow embedding custom PIB/TPM types
      
      Updated by Junxiao Shi almost 11 years ago
      
    
    Which commit are you talking about? I can't find this commit on Gerrit.
      
      Updated by Junxiao Shi almost 11 years ago
      
    
    I agree with this issue. Please be sure to put a comment above the function definition about why it shall stay in header, so that the same mistake won't happen in the future.
      
      Updated by Alex Afanasyev almost 11 years ago
      
    
    I will create a test case to handle the issue in the future.
      
      Updated by Alex Afanasyev almost 11 years ago
      
    
    - Status changed from New to In Progress
 - % Done changed from 0 to 100
 
      
      Updated by Alex Afanasyev almost 11 years ago
      
    
    Unfortunately, it wasn't possible to just move constructor into the header file. I did some refactoring of the PIB/TPM creation process.
      
      Updated by Alex Afanasyev almost 11 years ago
      
    
    - Status changed from In Progress to Closed
 
      
      Updated by Alex Afanasyev almost 11 years ago
      
    
    - Related to Task #2242: Pair up SecPublicInfo and SecTpm added
 
Actions