Clean up get_pgsm_query_id_hash()

[jeltz]

The function was unnecessarily convoluted so clean it up by making the parser easier to read, asserting against NULL input and wrapping the comment at the right width. Especially the code for handling whitespace was much more complicated than necessary.

PG-0

[codecov]

Codecov Report

❌ Patch coverage is 90.90909% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.57%. Comparing base (86edb37) to head (f9bf633).

Files with missing lines	Patch %	Lines
src/pg_stat_monitor.c	90.90%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #761      +/-   ##
==========================================
+ Coverage   88.33%   88.57%   +0.23%     
==========================================
  Files           3        3              
  Lines        1329     1330       +1     
  Branches      201      196       -5     
==========================================
+ Hits         1174     1178       +4     
  Misses         72       72              
+ Partials       83       80       -3     

☔ View full report in Codecov by Harness.
📢 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.

[dutow]

a SPACE /* */ SPACE b now emits a SPACE SPACE b, previously it was a SPACE b, which results in a different hash. Is this something that can cause issues on upgrade? Also this means that it is no longer strictly independent of comments.

[jeltz]

Ouch, good point. That was a stupid bug for me to introduce.