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: 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).