From: cyg Simple <cygsimple@gmail.com>
To: cygwin-patches@cygwin.com
Subject: Re: Trivial fix to last change
Date: Mon, 14 Dec 2015 13:38:00 -0000 [thread overview]
Message-ID: <566EC64D.5060802@gmail.com> (raw)
In-Reply-To: <20151214092507.GD3507@calimero.vinschen.de>
On 12/14/2015 4:25 AM, Corinna Vinschen wrote:
> On Dec 11 17:14, Ken Brown wrote:
>> cygwin1.dll doesn't build on x86 after the last commit (eed35ef). The
>> trivial patch attached fixes it.
>>
>> Ken
>
>> From 1cd61c54994b2ba6c6ec1d1f8f1249f5f8fd4af3 Mon Sep 17 00:00:00 2001
>> From: Ken Brown <kbrown@cornell.edu>
>> Date: Fri, 11 Dec 2015 17:08:28 -0500
>> Subject: [PATCH] Fix regparm attribute of fhandler_base::fstat_helper
>>
>> * winsup/cygwin/fhandler_disk_file.cc (fhandler_base::fstat_helper):
>> Align regparm attribute to declaration in fhandler.h.
>> ---
>> winsup/cygwin/ChangeLog | 5 +++++
>> winsup/cygwin/fhandler_disk_file.cc | 2 +-
>> 2 files changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
>> index 3c9804b..7079baa 100644
>> --- a/winsup/cygwin/ChangeLog
>> +++ b/winsup/cygwin/ChangeLog
>> @@ -1,3 +1,8 @@
>> +2015-12-11 Ken Brown <kbrown@cornell.edu>
>> +
>> + * fhandler_disk_file.cc (fhandler_base::fstat_helper): Align
>> + regparm attribute to declaration in fhandler.h.
>> +
>> 2015-12-10 Corinna Vinschen <corinna@vinschen.de>
>>
>> * path.h (class path_conv_handle): Use FILE_ALL_INFORMATION instead of
>> diff --git a/winsup/cygwin/fhandler_disk_file.cc b/winsup/cygwin/fhandler_disk_file.cc
>> index fe9dd03..1dd1b8c 100644
>> --- a/winsup/cygwin/fhandler_disk_file.cc
>> +++ b/winsup/cygwin/fhandler_disk_file.cc
>> @@ -428,7 +428,7 @@ fhandler_base::fstat_fs (struct stat *buf)
>> return res;
>> }
>>
>> -int __reg3
>> +int __reg2
>> fhandler_base::fstat_helper (struct stat *buf)
>> {
>> IO_STATUS_BLOCK st;
>> --
>> 2.6.2
>>
>
> Applied. I really should build on *both* architectures before applying
> a patch :-P
>
Would it have made more sense to test for architecture target?
--
cyg Simple
next prev parent reply other threads:[~2015-12-14 13:38 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-11 22:13 Ken Brown
2015-12-14 9:25 ` Corinna Vinschen
2015-12-14 13:38 ` cyg Simple [this message]
2015-12-14 15:06 ` Corinna Vinschen
2015-12-15 15:27 ` cyg Simple
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=566EC64D.5060802@gmail.com \
--to=cygsimple@gmail.com \
--cc=cygwin-patches@cygwin.com \
/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.
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).