public inbox for cygwin-cvs@sourceware.org help / color / mirror / Atom feed
From: Corinna Vinschen <corinna@sourceware.org> To: cygwin-cvs@sourceware.org Subject: [newlib-cygwin] Cygwin: fix a few comments mentioning Windows 7 or 8 Date: Sun, 4 Dec 2022 13:28:12 +0000 (GMT) [thread overview] Message-ID: <20221204132812.A3282384F480@sourceware.org> (raw) https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=a3af720dee13cb83f50406c4cddda635aca03e31 commit a3af720dee13cb83f50406c4cddda635aca03e31 Author: Corinna Vinschen <corinna@vinschen.de> AuthorDate: Tue Nov 15 21:50:13 2022 +0100 Commit: Corinna Vinschen <corinna@vinschen.de> CommitDate: Sun Dec 4 14:01:41 2022 +0100 Cygwin: fix a few comments mentioning Windows 7 or 8 Signed-off-by: Corinna Vinschen <corinna@vinschen.de> Diff: --- winsup/cygwin/fhandler/pipe.cc | 3 --- winsup/cygwin/fhandler/proc.cc | 1 - winsup/cygwin/local_includes/ntdll.h | 3 +-- winsup/cygwin/mount.cc | 6 +++--- winsup/cygwin/resource.cc | 3 +-- 5 files changed, 5 insertions(+), 11 deletions(-) diff --git a/winsup/cygwin/fhandler/pipe.cc b/winsup/cygwin/fhandler/pipe.cc index 608a78490879..94b85c34973b 100644 --- a/winsup/cygwin/fhandler/pipe.cc +++ b/winsup/cygwin/fhandler/pipe.cc @@ -1179,9 +1179,6 @@ cache_err: return get_query_hdl_per_process (name, ntfn); /* Since Win8 */ } -/* This function is faster than get_query_hdl_per_system(), however, - only works since Windows 8 because ProcessHandleInformation is not - suppoted by NtQueryInformationProcess() before Windows 8. */ HANDLE fhandler_pipe::get_query_hdl_per_process (WCHAR *name, OBJECT_NAME_INFORMATION *ntfn) diff --git a/winsup/cygwin/fhandler/proc.cc b/winsup/cygwin/fhandler/proc.cc index 24067f68751d..332bf2a7ea21 100644 --- a/winsup/cygwin/fhandler/proc.cc +++ b/winsup/cygwin/fhandler/proc.cc @@ -650,7 +650,6 @@ format_proc_cpuinfo (void *, char *&destbuf) char *buf = tp.c_get (); char *bufptr = buf; - //WORD num_cpu_groups = 1; /* Pre Windows 7, only one group... */ WORD num_cpu_per_group = __get_cpus_per_group (); cpu_num_p = wcpcpy (cpu_key, L"\\Registry\\Machine\\HARDWARE\\DESCRIPTION" diff --git a/winsup/cygwin/local_includes/ntdll.h b/winsup/cygwin/local_includes/ntdll.h index 0f2310882b2a..8c4d00883254 100644 --- a/winsup/cygwin/local_includes/ntdll.h +++ b/winsup/cygwin/local_includes/ntdll.h @@ -961,8 +961,7 @@ typedef struct _LDR_DATA_TABLE_ENTRY ULONG Flags; USHORT LoadCount; /* More follows. Left out since it's just not used. The aforementioned - part of the structure is stable from at least NT4 up to Windows 8, - including WOW64. */ + part of the structure is stable from at least NT4 up to Windows 11. */ } LDR_DATA_TABLE_ENTRY, *PLDR_DATA_TABLE_ENTRY; typedef struct _PEB_LDR_DATA diff --git a/winsup/cygwin/mount.cc b/winsup/cygwin/mount.cc index 63c9d2874121..9b40c4075aab 100644 --- a/winsup/cygwin/mount.cc +++ b/winsup/cygwin/mount.cc @@ -294,13 +294,13 @@ fs_info::update (PUNICODE_STRING upath, HANDLE in_vol) if (is_remote_drive ()) { /* Should be reevaluated for each new OS. Right now this mask is valid up - to Windows 8. The important point here is to test only flags indicating + to Windows 11. The important point here is to test only flags indicating capabilities and to ignore flags indicating a specific state of this volume. At present these flags to ignore are FILE_VOLUME_IS_COMPRESSED, FILE_READ_ONLY_VOLUME, and FILE_SEQUENTIAL_WRITE_ONCE. The additional filesystem flags supported since Windows 7 are also ignored for now. - They add information, but only on W7 and later, and only for filesystems - also supporting these flags, right now only NTFS. */ + They add information only for filesystems also supporting these flags, + right now only NTFS. */ #define GETVOLINFO_VALID_MASK (0x002701ffUL) #define TEST_GVI(f,m) (((f) & GETVOLINFO_VALID_MASK) == (m)) diff --git a/winsup/cygwin/resource.cc b/winsup/cygwin/resource.cc index aafc2b0b12fa..5ec436c2cfa4 100644 --- a/winsup/cygwin/resource.cc +++ b/winsup/cygwin/resource.cc @@ -218,8 +218,7 @@ __set_rlimit_as (unsigned long new_as_limit) /* If we already have a limit, we must not change it because that would potentially influence already running child processes. - Just try to create another, nested job. On systems prior to - Windows 8 / Server 2012 this will fail, but that's ok. */ + Just try to create another, nested job. */ while (new_as_id == 0) new_as_id = InterlockedIncrement (&job_serial_number); RtlInitUnicodeString (&uname,
reply other threads:[~2022-12-04 13:28 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=20221204132812.A3282384F480@sourceware.org \ --to=corinna@sourceware.org \ --cc=cygwin-cvs@sourceware.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: linkBe 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).