Hello,

I've got a couple of patches here.

- Missing copy constructor in WWL caused dtinfo to crash when setting browser geometry in preferences.

- clib complaining about invalid file handle fclosed in dtfile.

- Cosmic Flame Fractals screensaver lost its drawing code. There were unused variables involved, I guess the useful part went with them during some cleanups :)
>From 470d690a24645a91b1feb8b2c5d50c2f9a9935c8 Mon Sep 17 00:00:00 2001
From: alx <a...@fastestcode.org>
Date: Thu, 20 Feb 2014 14:07:43 +0100
Subject: wwl: fixed dtinfo segfault due to missing copy
 constructor in WArgList
Content-Length: 1218
Status: RO

---
 cde/programs/dtinfo/dtinfo/wwl/include/WWL/WArgList.h | 2 ++
 cde/programs/dtinfo/dtinfo/wwl/src/WArgList.C         | 8 ++++++++
 2 files changed, 10 insertions(+)

diff --git a/cde/programs/dtinfo/dtinfo/wwl/include/WWL/WArgList.h 
b/cde/programs/dtinfo/dtinfo/wwl/include/WWL/WArgList.h
index 61beed9..9f3e845 100644
--- a/cde/programs/dtinfo/dtinfo/wwl/include/WWL/WArgList.h
+++ b/cde/programs/dtinfo/dtinfo/wwl/include/WWL/WArgList.h
@@ -95,6 +95,8 @@ public :
                delete [] args;
       }
    }
+   
+   WArgList(const WArgList &r);
 
    inline ArgList      Args() const    { return args; }
    inline Cardinal     NumArgs() const { return num_args; }
diff --git a/cde/programs/dtinfo/dtinfo/wwl/src/WArgList.C 
b/cde/programs/dtinfo/dtinfo/wwl/src/WArgList.C
index 07d6583..363e7d8 100644
--- a/cde/programs/dtinfo/dtinfo/wwl/src/WArgList.C
+++ b/cde/programs/dtinfo/dtinfo/wwl/src/WArgList.C
@@ -162,6 +162,14 @@ WArgList::WArgList (String name, XtArgVal value, ...)
        va_end (argv);
 }
 
+WArgList::WArgList(const WArgList &r)
+{
+       num_args = r.num_args;
+       alloc_args = num_args;
+       args = new Arg[num_args];
+       memcpy(args, r.args, sizeof(Arg)*num_args);
+}
+
 /*?nodoc?*/
 Arg&
 WArgList::Grow (Cardinal n)
-- 
1.8.4

>From d9b56153719b57682684c4deae55345fdb093c01 Mon Sep 17 00:00:00 2001
From: alx <a...@fastestcode.org>
Date: Thu, 20 Feb 2014 14:14:46 +0100
Subject: dtfile: fclose called twice on same handle
Content-Length: 425
Status: RO

---
 cde/programs/dtfile/Directory.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cde/programs/dtfile/Directory.c b/cde/programs/dtfile/Directory.c
index 6a4607d..8f02d24 100644
--- a/cde/programs/dtfile/Directory.c
+++ b/cde/programs/dtfile/Directory.c
@@ -1683,7 +1683,6 @@ ReadDirectoryProcess(
 
      }
      XtFree( (char *)position_info );
-     fclose(fptr);
    }
 
    XtFree(full_directory_name);
-- 
1.8.4


>From aa5d9f4d49cd712f781e8997a1dab9b67bdfed21 Mon Sep 17 00:00:00 2001
From: alx <a...@fastestcode.org>
Date: Thu, 20 Feb 2014 14:28:12 +0100
Subject: dtscreen: cosmic flame fractals missing drawing code
Content-Length: 665

---
 cde/programs/dtscreen/flame.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/cde/programs/dtscreen/flame.c b/cde/programs/dtscreen/flame.c
index e4cfbca..48e7a0e 100644
--- a/cde/programs/dtscreen/flame.c
+++ b/cde/programs/dtscreen/flame.c
@@ -128,6 +128,8 @@ recurse(fs, x, y, l)
            return False;
 
        if (x > -1.0 && x < 1.0 && y > -1.0 && y < 1.0) {
+           fs->pts[fs->num_points].x = (int) ((fs->width / 2) * (x + 1.0));
+           fs->pts[fs->num_points].y = (int) ((fs->height / 2) * (y + 1.0));
            fs->num_points++;
            if (fs->num_points > MAXBATCH) {    /* point buffer size */
                XDrawPoints(dsp, fs->pwin->w, fs->pwin->gc, fs->pts,
-- 
1.8.4

------------------------------------------------------------------------------
Managing the Performance of Cloud-Based Applications
Take advantage of what the Cloud has to offer - Avoid Common Pitfalls.
Read the Whitepaper.
http://pubads.g.doubleclick.net/gampad/clk?id=121054471&iu=/4140/ostg.clktrk
_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to