feat(ffi)!: do not panic in visit_scan_metadata

[aleksandarskrbic]

What changes are proposed in this pull request?

Fixes #680

Avoid .unwrap() in visit_scan_metadata, replace return parameter to ExternResult<NullableCvoid>

Usage example in C:

  ExternResultNullableCvoid result = visit_scan_metadata(scan_metadata, context->engine, engine_context, scan_row_callback);
  if (result.tag != OkNullableCvoid) {
    printf("Error visiting scan metadata\n");
    exit(-1);
  }

How was this change tested?

Running read_table.c example

[codecov]

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.74%. Comparing base (be93add) to head (039edcf).

Files with missing lines	Patch %	Lines
ffi/src/scan.rs	0.00%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1526      +/-   ##
==========================================
- Coverage   84.75%   84.74%   -0.01%     
==========================================
  Files         122      122              
  Lines       33012    33015       +3     
  Branches    33012    33015       +3     
==========================================
  Hits        27980    27980              
- Misses       3660     3663       +3     
  Partials     1372     1372              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[nicklan]

This will break the examples so we'll need to fix those too.

[nicklan]

ohh, actually I see, it won't break them, but can we add a check that this succeeded in the examples, rather than just ignoring the result.