Bug 33083 - [Patch] kreversi aborts with out of memory on Alpha
Summary: [Patch] kreversi aborts with out of memory on Alpha
Status: CLOSED FIXED
Alias: None
Product: kreversi
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Other
: NOR normal
Target Milestone: ---
Assignee: Mario Weilguni
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-09-28 20:48 UTC by Dr. David Alan Gilbert
Modified: 2002-09-16 14:43 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Dr. David Alan Gilbert 2001-09-28 20:38:58 UTC 
(*** This bug was imported into bugs.kde.org ***)

Package: kreversi
Version: 2.2.0-final

Hi
  On Alpha kreversi has for a few weeks aborted with out of memory on Alpha
(Linux/Debian).

  I beleive the problem is cured by the following patch; it is caused by
loadChips using _size and _zoom in one of the functions it calls but they
are not initialised until just after.

Dave


--- board.cpp.predagFri Sep 28 21:34:33 2001
+++ board.cppFri Sep 28 21:34:47 2001
@@ -7612 +7613 @@
   for(int i = 0; i < CHIP_MAX; i++)
     chip[i] = 0;
 
+  _size = 32;
+  _zoom = 100;
+
   loadChips("chips.xpm");
 
   nopaint = FALSE;
   human = Score::BLACK;
-  _size = 32;
-  _zoom = 100;
   setStrength(4);
 
   connect(this SIGNAL(signalFieldClicked(int int))

--
 ---------------- Have a happy GNU millennium! ----------------------   
/ Dr. David Alan Gilbert    | Running GNU/Linux on Alpha68K| Happy  \ 
\ gro.gilbert @ treblig.org | MIPSx86ARM SPARC and HP-PA | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/
Comment 1 Waldo Bastian 2001-09-29 22:18:32 UTC 
Done thanks for your patch.

Cheers
Waldo

On Saturday 29 September 2001 03:41 am Dr. David Alan Gilbert wrote:
> Hi
>   Mario Weilguni asked me to forward my patch to kde-devel@kde.org since
> he no longer has time to maintain his packages; can someone check this and
> apply it please.
>
> It is filed as kde bug #33083
-- 
bastian@kde.org | SuSE Labs KDE Developer | bastian@suse.com