OpenCPN Version 2.4 Beta Build 423

[sinbad7]

Printing also works here on Vista....starting with a blanc page.
I was testing it printing to .pdf and as the first page was blanc,I
assumed the file was empty.

The page did not line up correcly though,and inch missing to the left and
a similar blanc space to the right and bottom of the map.

Tore

[bdbcat]

Hello testers!

Status report:

As you may see from Flyspray tasklist, there are enough "Solution Pending" items that I would normally be ready for a new Beta about now. And I am, except....

I would very much like to get to the bottom of these occasional Windows crashes on cm93 quilted chart dragging. Experience indicates that if we declare this issue to be an "unknowable glitch" in 423, and issue a new Beta, the problem will be temporarily masked. The probability of reappearance in the 2.5 Release or later would then be high, IME.

I have posted a debug procedure in this thread for those who can reproduce the fault. http://www.cruisersforum.com/forums/...tml#post677590

I ask that those who can do so, please try hard to reproduce and document this fault. To those who cannot, I beg your patience, please.

Thanks, and good luck
Dave

[nohal]

Dave,
don't have anything for the crashes, but have a little rendering issue while moving a waypoint around. FS#423 or something similar? Icon at http://www.cruisersforum.com/forums/...tml#post680199
Also this icon shows pretty clearly the reason for which I wrote the patch in FS#390

Pavel

[PjotrC]

With a somewhat heavy conscience, I did check whether the panning issue on appending to route reported in Forums - Post 679471">Feature-IRJDSUNE9932123321222xxeww-Requests-IRJDSUNE9932123321222xxeww-Thread has anything to do with RenderRouteLegInfo() I added there...

It seems it is yet another difference between the binary 423 distribution and the GIT source for 423. My tally runs as follows:

- the layers gpx files location
- Polish locale strings location
- HARMONIC.IDX contents
- append to route panning behaviour

[bdbcat]

PjotrC

Sorry, I am not following you.

Could you try to explain again?

Thanks
Dave

[PjotrC]

Dave...

It must be my slip on the GIT then. I have 3 running OCPN 423: downloaded installation, locally built opencpn and locally built PACKAGE setup. I recompiled and indeed I get the same behaviour now for Append Waypoint to Route in all three 423.

Thanks for your attention and efforts!

Piotr

[thenavigator]

Since Ubuntu 8.10 -D_FORTIFY_SOURCE=2 is enabled when compiling
with -O2 or higher[1]. OpenCPN "release" builds use -O3 and are
thus "fortified" [2]. The current (git repo) version causes a runtime
exception to be thrown around the potentially unsafe use of
strcpy in navutil.cpp:3041:

strcpy ( pOLE->OBJLName, sObj.mb_str() );

No compile time warning is given. Other occurrences of strcpy may
cause the same issue. I've had to define -U_FORTIFY_SOURCE as
CMAKE_CXX_FLAGS and CMAKE_C_FLAGS to disable the checks.

How to reproduce: Enable CM93 charts on release version built on
a recent Ubuntu (and likely other Linux variants). Restart application.

I am unfortunately not good enough a coder for replacing strcpy with
one of its safer alternatives.

[1] https://wiki.ubuntu.com/CompilerFlags
[2] Jakub Jelinek - [PATCH] Object size checking to prevent (some) buffer overflows