| Summary: | tdelibs: KDE4 is running | ||
|---|---|---|---|
| Product: | TDE | Reporter: | Darrell <darrella> |
| Component: | tdelibs | Assignee: | Timothy Pearson <kb9vqf> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | bugwatch, darrella |
| Priority: | P1 | ||
| Version: | R14.0.0 [Trinity] | ||
| Hardware: | Other | ||
| OS: | All | ||
| Compiler Version: | TDE Version String: | ||
| Application Version: | Application Name: | ||
| Attachments: | Patch to ignore NULL value and clean up code | ||
Patch pushed in GIT commit 3dfc6fd1. Resolved. |
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.