Refactoring

[artemgavrilov]

Description

Some refactoring:

1. Remove redundant argument from pgsm_create_hash_entry function
2. Move declarations that used in one place to that place (in the same file)
3. Little cleanup of main header file

Links

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.28%. Comparing base (486537b) to head (0a03600).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #635      +/-   ##
==========================================
- Coverage   85.44%   85.28%   -0.17%     
==========================================
  Files           3        3              
  Lines        1340     1339       -1     
  Branches      217      217              
==========================================
- Hits         1145     1142       -3     
- Misses         92       93       +1     
- Partials      103      104       +1     

☔ 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.

[AndersAstrand]

sence->sense in the first commit message.

I think the second commit is an improvement, but I think we should introduce guc.h and hash_query.h to make things even better than having those "sections" in the one h-file.

If you don't feel like doing that we should commit this as is though.

[artemgavrilov]

sence->sense in the first commit message.

I think the second commit is an improvement, but I think we should introduce guc.h and hash_query.h to make things even better than having those "sections" in the one h-file.

If you don't feel like doing that we should commit this as is though.

Adding separate header files it's what I planned to suggest next 😄

Also what do you think about moving C sources to /src directory?