* [PATCH 2/2] codesearch: generate_cxx: drop unused variables
2024-02-12 21:28 6% [PATCH 0/2] xap_helper C++ fixes Eric Wong
@ 2024-02-12 21:28 7% ` Eric Wong
0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2024-02-12 21:28 UTC (permalink / raw)
To: meta
We are just using the odd ref+deref (`${\...}') syntax and
don't need to calculate line numbers ourselves, nowadays.
---
lib/PublicInbox/CodeSearch.pm | 1 -
1 file changed, 1 deletion(-)
diff --git a/lib/PublicInbox/CodeSearch.pm b/lib/PublicInbox/CodeSearch.pm
index 48697cdc..1f95a726 100644
--- a/lib/PublicInbox/CodeSearch.pm
+++ b/lib/PublicInbox/CodeSearch.pm
@@ -101,7 +101,6 @@ sub qparse_new ($) {
}
sub generate_cxx () { # generates snippet for xap_helper.h
- my ($line, $file) = (__LINE__ + 2, __FILE__);
my $ret = <<EOM;
# line ${\__LINE__} "${\__FILE__}"
static NRP *code_nrp[${\scalar(@CODE_VMAP)}];
^ permalink raw reply related [relevance 7%]
* [PATCH 0/2] xap_helper C++ fixes
@ 2024-02-12 21:28 6% Eric Wong
2024-02-12 21:28 7% ` [PATCH 2/2] codesearch: generate_cxx: drop unused variables Eric Wong
0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2024-02-12 21:28 UTC (permalink / raw)
To: meta
I suppose -O2 build times isn't the worst thing for release
users even though I can't stand it while hacking...
Eric Wong (2):
xap_helper_cxx: -O2 optimize read-only files by default
codesearch: generate_cxx: drop unused variables
lib/PublicInbox/CodeSearch.pm | 1 -
lib/PublicInbox/XapHelperCxx.pm | 1 +
2 files changed, 1 insertion(+), 1 deletion(-)
^ permalink raw reply [relevance 6%]
Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2024-02-12 21:28 6% [PATCH 0/2] xap_helper C++ fixes Eric Wong
2024-02-12 21:28 7% ` [PATCH 2/2] codesearch: generate_cxx: drop unused variables Eric Wong
Code repositories for project(s) associated with this public inbox
https://80x24.org/public-inbox.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).