diff options
author | Johannes Schickel | 2009-09-15 22:16:55 +0000 |
---|---|---|
committer | Johannes Schickel | 2009-09-15 22:16:55 +0000 |
commit | eec86941f2ce35e4826abf218c5ad6f35051310b (patch) | |
tree | cd552c1720d5090b5829b7af0279a25892179bd2 | |
parent | 62ea7be0cfa462b5aab6a1c63fce67e4499c62a6 (diff) | |
download | scummvm-rg350-eec86941f2ce35e4826abf218c5ad6f35051310b.tar.gz scummvm-rg350-eec86941f2ce35e4826abf218c5ad6f35051310b.tar.bz2 scummvm-rg350-eec86941f2ce35e4826abf218c5ad6f35051310b.zip |
Add link to gcc bug report, in the comment for the gcc aimed workaround for AMD64.
svn-id: r44112
-rw-r--r-- | engines/cine/pal.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/engines/cine/pal.cpp b/engines/cine/pal.cpp index 60c58ac27c..ea8f6fbaab 100644 --- a/engines/cine/pal.cpp +++ b/engines/cine/pal.cpp @@ -223,8 +223,11 @@ Palette &Palette::saturatedAddColor(Palette& output, byte firstIndex, byte lastI // even though the struct is only 3 bytes, resulting in an invalid read, when accessing indices // 14 and 15 of 16 color palettes. // - // To work around this issue, we added an temporary variable, which will have padding so + // To work around this issue, we added an temporary variable, which will have padding, so // the 8 byte read (which is done when passing src) is assured to be in a valid memory area. + // + // For more information about this gcc specific problem, you can read up on the following bug + // report: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36043 const Color src = _colors[i]; output._colors[i] = saturatedAddColor(src, r, g, b); } |