Improve building with VS2015 by evpobr · Pull Request #2464 · notepad-plus-plus/notepad-plus-plus

evpobr · 2016-10-22T11:12:21Z

Now there is complete VS2015 solution in \Projects\VS2015 folder - Notepad++.sln. You can use it to build SciLexer.dll, notepad++.exe and related binaries in one step with Visual Studio 2015.

Fixes warning D9002: ignoring unknown option '/arch:IA32'

MAPJe71 · 2016-10-23T11:58:23Z

Proposal: in addition to NppShell project add projects for IExplorerShell, xmlApiSorter and xmlUpdater.

MAPJe71 · 2016-10-24T17:03:41Z

FYI: Although the source files are still part the repository, ChangeIcon is no longer used.

This reverts commit d262a0b.

donho · 2016-10-28T12:23:49Z

You can use it to build SciLexer.dll and notepad++.exe in one step with Visual Studio 2015.

With or without RegEx of boost ?

evpobr · 2016-10-28T12:32:12Z

Boost RegEx with NuGet package.

chcg · 2016-10-30T11:06:32Z

PowerEditor/src/MISC/Common/Common.h

+#define generic_fopen _tfopen
+#define generic_fgets _fgetts
+#define generic_stat _tstat
+#ifdef UNICODE


non UNICODE support was dropped some time ago, why should it come back?

, see Revision: 5c48259
Author: Don Ho don.h@free.fr
Date: 26.12.2015 01:17:01
Message:
Clean up

Remove unnecessary directive.

Modified: PowerEditor/src/MISC/Common/Common.h

chcg · 2016-10-30T11:10:49Z

Also precompiled header already exists and were dropped in favour of multiprocessor build as far as I remember around here:

Revision: b35e759
Author: Don Ho don.h@free.fr
Date: 31.05.2015 15:57:17
Message:
[UPDATE] Unprecompile headers

Modified: PowerEditor/src/MISC/Common/Common.h
Modified: PowerEditor/src/MISC/Common/precompiledHeaders.h
...

Do you have measurements for a speedup with precompiled headers?

evpobr · 2016-10-30T14:30:53Z

non UNICODE support was dropped some time ago, why should it come back?

@chcg: @MAPJe71 wants xmlApiSorter and xmlUpdater projects, they are ANSI console programs. To build them i need ANSI versions of generic_XXX functions.

Also precompiled header already exists and were dropped in favour of multiprocessor build as far as I remember around here:

Interesting question, documentation says these options are incompatible. I was not sure so i run some tests. Here are the results:

/MP	/Yc	Time
yes	yes	45830 ms
no	yes	68709 ms
yes	no	61363 ms
no	no	119763 ms

Surprisely, they work together in VS2015 just fine. I will enable them for other projects. Did not get any warnings.

MAPJe71 · 2016-10-30T18:46:06Z

@evpobr & @chcg
Wasn't aware that xmlApiSorter and xmlUpdater are still ANSI. This make me wonder if they are still used.

@donho
Are these console tools still in use?
When no longer in use: wouldn't it be better to remove them from the repository?
When still in use: should they be converted to Unicode tools?

chcg · 2016-11-01T21:34:36Z

See
https://blogs.msdn.microsoft.com/vcblog/2016/10/26/recommendations-to-speed-c-builds-in-visual-studio/
and
https://randomascii.wordpress.com/2014/03/22/make-vc-compiles-fast-through-parallel-compilation/
for optimization hints and mp build parallelization blockage.

Release 5.5.6 (https://www.scintilla.org/scintilla556.zip) Released 2 April 2025. Disallow changing case of protected text. Bug notepad-plus-plus#2463. Return enumeration type from MarkerSymbolDefined to match MarkerDefine. Bug notepad-plus-plus#2469. On Win32, use DirectWrite for autocompletion lists when DirectWrite chosen for document text. On Win32, optimize case-insensitive DBCS search to be around 5 times faster by using 64K memory to cache folding data for each DBCS code page used. On Win32, fix a crash with bidirectional text. When using Visual C++ through nmake, fix building for ARM64. Feature notepad-plus-plus#1546. On Qt, draw clipped UTF-8 text correctly. Bug notepad-plus-plus#2464. On Qt, avoid a dwell start when the mouse is moved outside the Scintilla widget. Bug notepad-plus-plus#2466. On Qt, autoCompleteSelection converts from local encoding when not in Unicode mode. Bug notepad-plus-plus#2465. Release 5.4.4 (https://www.scintilla.org/lexilla544.zip) Released 2 April 2025. Fix building for ARM64. Pull request notepad-plus-plus#308.

Release 5.5.6 (https://www.scintilla.org/scintilla556.zip) Released 2 April 2025. * Disallow changing case of protected text. Bug #2463. * Return enumeration type from MarkerSymbolDefined to match MarkerDefine. Bug #2469. * On Win32, use DirectWrite for autocompletion lists when DirectWrite chosen for document text. * On Win32, optimize case-insensitive DBCS search to be around 5 times faster by using 64K memory to cache folding data for each DBCS code page used. * On Win32, fix a crash with bidirectional text. * When using Visual C++ through nmake, fix building for ARM64. Feature #1546. * On Qt, draw clipped UTF-8 text correctly. Bug #2464. * On Qt, avoid a dwell start when the mouse is moved outside the Scintilla widget. Bug #2466. * On Qt, autoCompleteSelection converts from local encoding when not in Unicode mode. Bug #2465. Release 5.4.4 (https://www.scintilla.org/lexilla544.zip) Released 2 April 2025. * Fix building for ARM64. Pull request #308. Close #16373

evpobr added 20 commits October 22, 2016 15:20

Add more VS2015 .gitignore rules

03d5f8c

SciLexer: add VS2015 solution

2829848

Improve N++ VS2015 project

476a8bf

SciLexer: add WinXP support

1b6ac03

VS2015: change debug formats to default ones

9a54be6

VS2015: configure output path

f3e62b3

VS2015: do not show library search progress

b9b9ff1

VS2015: rename project name to notepad++

84062fe

VS2015: use default optimization for N++

8d6f668

VS2015: use incremental linking for Debug

3a1f1fd

VS2015: speep up builds for N++

4e5c995

VS2015: copy XML files to output directory

c96d159

VS2015: add filter to N++ project

44c5cbd

VS2015: remove unneeded switch

fe05f52

Fixes warning D9002: ignoring unknown option '/arch:IA32'

VS2015: enable LTG for N++

e507838

VS2015: N++ project small fixes

12a05b5

VS2015: add precompiled header

d01e5aa

VS2015: move to Projects\VS2015 folder

b7a5c86

VS2015: N++ project rebased on VS2013 project

9646697

VS2015: add NppShell project

6f9c565

VS2015: add ChangeIcon project

d262a0b

evpobr added 3 commits October 24, 2016 22:29

Revert "VS2015: add ChangeIcon project"

47b617c

This reverts commit d262a0b.

Fix Common.h for non-unicode builds

0718857

VS2015: add xmlUpdater project

5a7bbe9

evpobr added 2 commits October 29, 2016 13:36

VS2015: add sorter (xmlApiSorter) project

2a06a63

VS2015: add nppIExplorerShell project

764ecb7

evpobr added 8 commits October 29, 2016 13:51

VS2015: enable precompiler headers for NppShell

c1930ee

VS2015: enable precompiled headers for SciLexer

f0e799e

VS2015: enable precompiled headers for nppIExplorerShell

ded90bd

VS2015: enable precompiled headers for sorter

064e520

VS2015: enable precompiled headers for xmlUpdater

796d793

VS2015: fix copy task broken in 9646697

4b7c753

VS2015: use static runtime library

63c59af

VS2015: set W3 warning level for all projects

50086d1

chcg reviewed Oct 30, 2016

View reviewed changes

evpobr closed this May 11, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Improve building with VS2015#2464

Improve building with VS2015#2464
evpobr wants to merge 34 commits intonotepad-plus-plus:masterfrom
evpobr:vs2015

evpobr commented Oct 22, 2016 •

edited

Loading

Uh oh!

MAPJe71 commented Oct 23, 2016

Uh oh!

MAPJe71 commented Oct 24, 2016

Uh oh!

donho commented Oct 28, 2016

Uh oh!

evpobr commented Oct 28, 2016 •

edited

Loading

Uh oh!

chcg Oct 30, 2016

Uh oh!

chcg commented Oct 30, 2016 •

edited

Loading

Uh oh!

evpobr commented Oct 30, 2016 •

edited

Loading

Uh oh!

MAPJe71 commented Oct 30, 2016

Uh oh!

chcg commented Nov 1, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

evpobr commented Oct 22, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MAPJe71 commented Oct 23, 2016

Uh oh!

MAPJe71 commented Oct 24, 2016

Uh oh!

donho commented Oct 28, 2016

Uh oh!

evpobr commented Oct 28, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chcg Oct 30, 2016

Choose a reason for hiding this comment

Remove unnecessary directive.

Uh oh!

chcg commented Oct 30, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

evpobr commented Oct 30, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MAPJe71 commented Oct 30, 2016

Uh oh!

chcg commented Nov 1, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

evpobr commented Oct 22, 2016 •

edited

Loading

evpobr commented Oct 28, 2016 •

edited

Loading

chcg commented Oct 30, 2016 •

edited

Loading

evpobr commented Oct 30, 2016 •

edited

Loading