[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gnash-commit
Subject:    [Gnash-commit] [patch #8942] Patches to improve number value handling in ExternalInterface
From:       Nutchanon Wetchasit <INVALID.NOREPLY () gnu ! org>
Date:       2016-03-13 12:59:59
Message-ID: 20160313-195959.sv82809.41145 () savannah ! gnu ! org
[Download RAW message or body]

Follow-up Comment #1, patch #8942 (project gnash):

Current libgnashplugin marshall `double` value sent from NPRuntime
to `<number>` data using default iostream with no specific precision
<https://savannah.gnu.org/bugs/?47280#comment4>
specified. This causes precision loss as not enough decimal digits
were used to represent the value.

Patch 2 of 2:
See the attached
`0002_plugin-externalinterface-number-marshall-enough-digits.patch`.

This patch explicitly specifies iostream floating point precision to
`std::numeric_limits<double>::max_digits10` (C++ 11) which is
enough significands to accurately reproduce the original `double` value
(provided that receiver end's parser library implements the correct
rounding).

This partially fixes bug #47280.

Gnash: 0.8.11dev (patched against git a72afa5 9-Mar-2016) NPAPI
Browser: Iceweasel 10.0.12 (debian)
System: Debian GNU/Linux 7.0 Wheezy i386


(file #36619)
    _______________________________________________________

Additional Item Attachment:

File name: 0002_plugin-externalinterface-number-marshall-enough-digits.patch
Size:0 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8942>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/


_______________________________________________
Gnash-commit mailing list
Gnash-commit@gnu.org
https://lists.gnu.org/mailman/listinfo/gnash-commit
[prev in list] [next in list] [prev in thread] [next in thread]