By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1200 - tdelibs: KDE4 is running
Summary: tdelibs: KDE4 is running
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdelibs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other All
: P1 minor
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2012-09-05 20:38 CDT by Darrell
Modified: 2013-03-02 17:58 CST (History)
2 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Patch to ignore NULL value and clean up code (2.52 KB, patch)
2012-09-05 20:38 CDT, Darrell 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2012-09-05 20:38:25 CDT 
Created attachment 807 [details]
Patch to ignore NULL value and clean up code

When kdebug area 7020 is enabled, the following message appears in the xsession log:

KDE4 is running.

The test for displaying this message is to obtain the value of environment variable KDE_SESSION_VERSION, which on a system without KDE4 will not exist. Apparently TQString interprets NULL as a value, which in the subsequent test is seen as being greater than "4" and results in displaying the message.

Attached is a patch to ignore a NULL value.

The patch adds two additional messages to support the meaning of "KDE4 is running."

The patch renames the boolean variable kde_running to tde_running to help avoid confusion with the other code regarding KDE4.
Comment 1 Darrell 2013-03-02 17:58:34 CST 
Patch pushed in GIT commit 3dfc6fd1. Resolved.