Remove duplicated *.ncb and *.aps rules from VisualStudio.gitignore

Both rules were already present under "Visual C++ cache files", where they
have been since the template was created in 2010 (590e423). A second copy
was added in 2019 (e285e4b) under a new "Visual Studio 6 technical files"
heading, whose commit described them as "their purpose is unknown for me".

Beyond being duplicates, the second heading is inaccurate. Neither file is
specific to Visual Studio 6:

- *.ncb is the Visual C++ IntelliSense database, used through Visual Studio
  2008 and replaced by the SDF database in 2010. The template already lists
  *.sdf two lines below *.ncb in the "Visual C++ cache files" section, so
  the pair belongs together there.
- *.aps is the binary form of the resource script used by the resource
  editor. Microsoft documents it for msvc-140 through msvc-180 (Visual
  Studio 2015 to 2022) and states plainly that it should not be in source
  control.

The genuinely VS6-specific rules added by that commit (*.vbw, *.dsw, *.dsp)
are untouched. Behaviour is unchanged: both patterns are still ignored, once
each, under the heading that describes them correctly.

VisualStudio.gitignore

@@ -328,10 +328,6 @@ node_modules/
 *.dsw
 *.dsp
 
-# Visual Studio 6 technical files
-*.ncb
-*.aps
-
 # Visual Studio LightSwitch build output
 **/*.HTMLClient/GeneratedArtifacts
 **/*.DesktopClient/GeneratedArtifacts