Re: Avoid mix char with bool type in comparisons

Em sex., 7 de out. de 2022 às 13:32, Robert Haas <robertmhaas(at)gmail(dot)com>
escreveu:

> On Fri, Oct 7, 2022 at 11:40 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > FWIW, I did take a look at this code, and I don't see any bug.
> > The entryRes[] array entries are indeed GinTernaryValue, but it's
> > obvious by inspection that matchPartialInPendingList only returns
> > true or false, therefore collectMatchesForHeapRow also only deals
> > in true or false, never maybe. I do not think changing
> > matchPartialInPendingList to return ternary would be an improvement,
> > because then it'd be less obvious that it doesn't deal in maybe.
>
> I mean if the code isn't buggy, I'm glad, but I think there should
> have been more substantial grounds for getting you to spend time
> looking at it. It's not asking too much for people to produce a
> non-zero amount of evidence that the thing they are worried about is
> actually a problem.
>
Sorry if you think this is all just a waste of time.

I think that while the current code has no real bugs, that doesn't mean it
doesn't have readability and style issues.
And that not being able to produce tests should not be an impediment to
improving the current code.
I believe I have contributed much more than changing "fo" to "of" in
comments.

Right now I have:
02/09/2022 15:58 593 0001-fix-typo-isnan-test-geo_ops.patch
02/09/2022 15:57 7.746 0001-fix-wrong-isnan-test-geo_ops.patch
11/07/2022 09:39 2.271
0001-Promove-unshadowing-of-two-variables-PGPROC-type.patch
11/07/2022 09:39 2.939
0001-Reduce-Wsign-compare-warnings-from-clang-12-compiler.patch
11/07/2022 09:39 15.377
0001-Refactoring-strlen-comparisons-with-zero.patch
02/09/2022 09:06 6.711
0002-avoid-small-issues-brin_minmax_multi.patch
11/07/2022 09:39 6.402
001-aset-reduces-memory-consumption.patch
01/07/2022 12:53 2.111 001-avoid-unecessary-MemSet-calls.patch
11/07/2022 09:39 59.662 001-improve-executor.patch
11/07/2022 09:39 69.155 001-improve-getsnapshot.patch
11/07/2022 09:39 15.465 001-improve-memory.patch
11/07/2022 09:39 24.130 001-improve-scability-procarray.patch
11/07/2022 09:39 74.693 001-improve-scaling.patch
22/05/2022 13:23 6.579 001-improve-sort.patch
11/07/2022 09:39 280.420 001-improve-table-open.patch
11/07/2022 09:39 11.451 001-reduces-memory-consumption.patch
11/07/2022 09:39 8.516
002-generation-reduces-memory-consumption.patch
11/07/2022 09:39 6.025
003-aset-reduces-memory-consumption.patch
11/07/2022 09:39 8.966
004-generation-reduces-memory-consumption_BUG.patch
04/09/2022 18:28 12.095 all.patch
05/10/2022 09:41 2.048 all2.patch
20/09/2022 10:59 1.513 all_20_09_2022.patch
09/10/2020 11:42 673 avoid_dereferencing_null_pointer.patch
29/09/2022 20:39 437 avoid_useless_reassign_lgosegno.patch
29/09/2022 20:43 418
avoid_useless_retesting_log_min_duration.patch
29/09/2022 20:44 625 avoid_useless_var_record.patch
11/07/2022 09:39 32.180 FAST-001-improve-scability.patch
11/07/2022 09:39 51.453 FAST-001-improve-sort.patch
11/07/2022 09:39 62.491 FAST2-001-improve-sort.patch
04/10/2022 08:22 493
fix_declaration_volatile_signal_pg_test_fsync.patch
29/09/2022 20:45 484
fix_declaration_volatile_signal_var.patch
25/08/2020 12:19 1.087 fix_dereference_null_statscmds.patch
26/06/2020 11:26 1.526 fix_null_deference_pquery.patch
28/08/2020 15:53 537 fix_null_memcmp_call.patch
25/08/2020 14:53 541 fix_possible_overflow_executils.patch
25/08/2020 14:17 757 fix_possible_overflow_nodeagg.patch
05/09/2020 10:45 14.049 fix_redudant_init.patch
05/09/2020 10:35 933
fix_redudant_initialization_arrayfuncs.patch
05/09/2020 10:47 2.403
fix_redudant_initialization_bklno_hash.patch
05/09/2020 10:07 793
fix_redudant_initialization_firstmissingnum_heaptuple.patch
05/09/2020 10:36 362
fix_redudant_initialization_formatting.patch
05/09/2020 10:08 406
fix_redudant_initialization_offsetnumber_gistutil.patch
05/09/2020 10:25 851
fix_redudant_initialization_parse_utilcmd.patch
05/09/2020 10:29 742
fix_redudant_initialization_procarray.patch
05/09/2020 10:30 604 fix_redudant_initialization_spell.patch
05/09/2020 10:16 1.157
fix_redudant_initialization_status_nbtsearch.patch
05/09/2020 10:21 537
fix_redudant_initialization_storage.patch
05/09/2020 10:28 531
fix_redudant_initialization_syslogger.patch
05/09/2020 10:31 878
fix_redudant_initialization_to_tsany.patch
05/09/2020 10:36 452 fix_redudant_initialization_tsrank.patch
05/09/2020 10:38 1.324
fix_redudant_initialization_tuplesort.patch
05/09/2020 10:34 428
fix_redudant_initialization_wparser_def.patch
05/09/2020 10:18 797 fix_redudant_prefix_spgtextproc.patch
05/09/2020 10:19 834 fix_redudant_waits_xlog.patch
25/08/2020 15:48 2.319 fix_unchecked_return_spi_connect.patch
09/10/2020 09:15 420 fix_uninitialized_var_flag_spell.patch
09/09/2022 11:25 68.543 fprintf_fixes.patch
09/09/2020 09:17 13.805 getsnapshotdata.patch
27/09/2022 16:05 4.083 head_27_09_2022.patch
24/08/2020 19:31 21.023 hugepage.patch
14/05/2022 20:32 6.545 improve_sort.patch
15/09/2022 11:50 6.327 patchs_16_09_2022.patch
05/10/2022 14:30 15.376 postgres_05_10_2022.patch
11/07/2022 16:25 2.068 postgres_executor.patch
29/06/2022 11:01 29.995 postgres_sort.patch
07/09/2020 22:07 25.449 prefetch.patch
14/09/2020 10:22 19.919 setvbuf.patch
14/09/2020 14:36 19.919 setvfbuf.patch
05/09/2022 13:40 7.857 string_fixes.patch
11/07/2022 09:39 34.130 strlen.patch
05/10/2022 09:42 2.048 style_use_compatible_var_type.patch
28/08/2020 10:19 5.155 unloop_toast_tuple_init.patch
14/09/2022 20:00 3.237
use-heapalloc-instead-deprecated-localalloc.patch
11/09/2020 11:47 3.733
v1-0001-simplified_read_binary_file.patch
07/07/2022 15:22 106.755
v1-0001-WIP-Replace-MemSet-calls-with-struct-initialization.patch
11/07/2022 09:39 14.918 v1-001-improve-memory.patch
28/05/2022 08:45 24.989 v1-001-improve-scability-procarray.patch
11/07/2022 09:39 14.972 v1-001-improve-scaling.patch
09/09/2022 11:26 68.543 v1-fprintf_fixes.patch
05/09/2022 21:42 20.586 v1-string_fixes.patch
11/07/2022 09:39 34.469 v10-001-improve-scability.patch
11/07/2022 09:39 32.229 v11-001-improve-scability.patch
11/07/2022 09:39 36.060 v12-001-improve-scability.patch
11/07/2022 09:39 53.064 v13-001-improve-scability.patch
11/07/2022 09:39 48.123 v14-001-improve-scability.patch
11/07/2022 09:39 35.443 v15-001-improve-scability.patch
09/08/2022 15:56 95.542
v2-0001-Improve-performance-of-and-reduce-overheads-of-me.patch
11/09/2020 16:58 4.228
v2-0001-simplified_read_binary_file.patch
11/07/2022 16:03 106.755
v2-0001-WIP-Replace-MemSet-calls-with-struct-initialization.patch
11/07/2022 09:39 17.894 v2-001-improve-memory.patch
11/07/2022 09:39 349.497 v2-001-improve-scability-procarray.patch
11/07/2022 09:39 69.227 v2-001-improve-scaling.patch
11/07/2022 09:39 18.709
v2-002-generation-reduces-memory-consumption.patch
11/07/2022 09:39 42.893 v2-002-improve-sort.patch
05/09/2022 23:16 52.064 v2-string_fixes.patch
11/09/2020 18:38 4.047
v3-0001-simplified_read_binary_file.patch
01/08/2022 13:52 26.670
v3-0001-WIP-Replace-MemSet-calls-with-struct-initialization.patch
11/07/2022 09:39 349.781 v3-001-improve-scability-procarray.patch
11/07/2022 09:39 45.707 v3-002-improve-sort.patch
05/09/2022 08:34 7.510
v3_avoid_referencing_out_of_bounds_array_elements.patch
15/09/2020 14:29 4.306
v4-0001-simplified_read_binary_file.patch
11/07/2022 09:39 352.181 v4-001-improve-scability.patch
11/07/2022 09:39 51.453 v4-002-improve-sort.patch
11/07/2022 09:39 354.611 v5-001-improve-scability.patch
11/07/2022 09:39 51.453 v5-002-improve-sort.patch
11/07/2022 09:39 355.739 v6-001-improve-scability.patch
11/07/2022 09:39 61.904 v6-002-improve-sort.patch
11/07/2022 09:39 13.547 v7-001-improve-scability.patch
11/07/2022 09:39 62.491 v7-002-improve-sort.patch
11/07/2022 09:39 27.800 v8-001-improve-scability.patch
11/07/2022 09:39 33.358 v9-001-improve-scability.patch
27/06/2020 11:17 7.754 windows_fixes_v1.patch

And it could contribute much, much more.

regards,
Ranier Vilela