solana: some clean up

Author: kcsongor

solana/migrations/deploy.ts

@@ -5,7 +5,6 @@ use anchor_lang::prelude::*;
 /// A sibling on another chain. Stored in a PDA seeded by the chain id.
 pub struct EndpointSibling {
     pub bump: u8,
-    // TODO: variable address length?
     pub address: [u8; 32],
 }
 

solana/programs/example-native-token-transfers/src/endpoints/accounts/sibling.rs

@@ -28,6 +28,9 @@ pub struct ReceiveMessage<'info> {
     )]
     pub sibling: Account<'info, EndpointSibling>,
 
+    // TODO: Consider using VaaAccount from wormhole-solana-vaa crate. Using a zero-copy reader
+    // will allow this instruction to be generic (instead of strictly specifying NativeTokenTransfer
+    // as the message type).
     #[account(
         // check that the messages is targeted to this chain
         constraint = vaa.message().manager_payload.payload.to_chain == config.chain_id @ NTTError::InvalidChainId,