Simplify some conditions related to [LW]Lock in generate-wait_event_types.pl
authorMichael Paquier <[email protected]>
Thu, 13 Jul 2023 00:09:04 +0000 (09:09 +0900)
committerMichael Paquier <[email protected]>
Thu, 13 Jul 2023 00:09:04 +0000 (09:09 +0900)
The first check on the enum values was not necessary as the values set
in wait_event_names.txt for the classes LWLock and Lock were able to
satisfy the check.  The second check when generating the C and header
files is now based on a match of the class names, making it simpler to
understand.

Author: Masahiro Ikeda, Michael Paquier
Discussion: https://postgr.es/m/eaf82a85c0ef1b55dc3b651d3f7b867a@oss.nttdata.com

src/backend/utils/activity/generate-wait_event_types.pl

index 6d1a2af42aedaafad6b3d1e81544a813f35158e5..75cddf6fa268a80f091a062adea389298e5746c6 100644 (file)
@@ -85,13 +85,8 @@ foreach my $line (@lines_sorted)
    my $trimmedwaiteventname = $waiteventenumname;
    $trimmedwaiteventname =~ s/^WAIT_EVENT_//;
 
-   # An exception is required for LWLock and Lock as these don't require
-   # any C and header files generated.
    die "wait event names must start with 'WAIT_EVENT_'"
-     if ( $trimmedwaiteventname eq $waiteventenumname
-       && $waiteventenumname !~ /^LWLock/
-       && $waiteventenumname !~ /^Lock/);
-   $continue = ",\n";
+     if ($trimmedwaiteventname eq $waiteventenumname);
    push(@{ $hashwe{$waitclassname} }, @waiteventlist);
 }
 
@@ -141,8 +136,8 @@ if ($gen_code)
        # Don't generate .c and .h files for LWLock and Lock, these are
        # handled independently.
        next
-         if ( $waitclass =~ /^WaitEventLWLock$/
-           || $waitclass =~ /^WaitEventLock$/);
+         if ( $waitclass eq 'WaitEventLWLock'
+           || $waitclass eq 'WaitEventLock');
 
        my $last = $waitclass;
        $last =~ s/^WaitEvent//;