From: Peter Stephenson <p.w.stephenson@ntlworld.com>
To: Clinton Bunch <cdb_zsh@zentaur.org>, zsh-workers@zsh.org
Subject: Re: current git Fails to compile on Solaris 11 x86
Date: Wed, 20 Mar 2024 16:15:14 +0000 (GMT) [thread overview]
Message-ID: <615589318.5857173.1710951314600@mail.virginmedia.com> (raw)
In-Reply-To: <7e9a2bfd-3a91-4c19-ad11-0397a839be4f@zentaur.org>
> On 20/03/2024 15:05 GMT Clinton Bunch <cdb_zsh@zentaur.org> wrote:
>
>
> On 3/20/2024 9:54 AM, Bart Schaefer wrote:
> > On Wed, Mar 20, 2024 at 5:22 AM Clinton Bunch <cdb_zsh@zentaur.org> wrote:
> >> On 3/20/2024 05:19, Peter Stephenson wrote:
> >>> Let's just rename it and the others in that area.
> >>>
> >> I'm getting the same error with that patch applied. What I saw on the
> >> lines it errored on were return statements returning the value of a
> >> function in a wrapper function defined to return void. Perhaps, simply
> >> removing the return keyword?
> > Yes, just removing the "return" from the void function is all that's needed.
> What I was unsure of was whether removing the return or changing the
> type of the wrapper function was the appropriate fix. That would
> require more familiarity with the code than I have (or at this time want
> to put in the effort to get) to answer. I mostly wanted to make others
> aware of the problem.
The wrapper function just fits a generic form, so should stay void.
It's as simple as it looks. (I guess the line numbers were out in the
code I was looking at so I misidentified what it was complaining about.)
diff --git a/Src/Modules/hlgroup.c b/Src/Modules/hlgroup.c
index 6382f32..230f453 100644
--- a/Src/Modules/hlgroup.c
+++ b/Src/Modules/hlgroup.c
@@ -137,7 +137,7 @@ getpmesc(UNUSED(HashTable ht), const char *name)
static void
scanpmesc(UNUSED(HashTable ht), ScanFunc func, int flags)
{
- return scangroup(func, flags, 0);
+ scangroup(func, flags, 0);
}
/**/
@@ -151,7 +151,7 @@ getpmsgr(UNUSED(HashTable ht), const char *name)
static void
scanpmsgr(UNUSED(HashTable ht), ScanFunc func, int flags)
{
- return scangroup(func, flags, 1);
+ scangroup(func, flags, 1);
}
static struct paramdef partab[] = {
next prev parent reply other threads:[~2024-03-20 16:15 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-20 4:41 Clinton Bunch
2024-03-20 9:30 ` Peter Stephenson
2024-03-20 10:19 ` Peter Stephenson
2024-03-20 12:21 ` Clinton Bunch
2024-03-20 14:54 ` Bart Schaefer
2024-03-20 15:05 ` Clinton Bunch
2024-03-20 16:15 ` Peter Stephenson [this message]
2024-03-20 22:26 ` Oliver Kiddle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=615589318.5857173.1710951314600@mail.virginmedia.com \
--to=p.w.stephenson@ntlworld.com \
--cc=cdb_zsh@zentaur.org \
--cc=zsh-workers@zsh.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
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 NNTP newsgroup(s).