Deprecate is_builtin_key_or_sysvar function

[jstarry]

Problem

The public is_builtin_key_or_sysvar function uses static lists of sdk keys and should be removed in favor of using the new ReservedAccountKeys struct.

Summary of Changes

• Deprecated is_builtin_key_or_sysvar
• Updated is_maybe_writable methods to accept an optional set of reserved keys. The reason it's optional is so that clients can opt out of providing the reserved key set if they don't care about it. Note that this method isn't to be used by the runtime so there's no need to enforce that a reserved key set is passed.

Breaking Changes

• v0::Message::is_maybe_writable has an additional param for passing reserved account keys
• VersionedMessage::is_maybe_writable has an additional param for passing reserved account keys

Fixes #

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 97.43590% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 81.8%. Comparing base (2c0bea0) to head (f86c195).
Report is 19 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master     #788     +/-   ##
=========================================
- Coverage    81.9%    81.8%   -0.2%     
=========================================
  Files         851      851             
  Lines      231480   231678    +198     
=========================================
- Hits       189728   189580    -148     
- Misses      41752    42098    +346     

[CriesofCarrots]

Generally lgtm. Just one small thought to make is_account_maybe_reserved() more readable to my brain.

[joncinque]

Looks good to me overall! Just a couple of questions