Bug 94337 - GUI reacts very slow
Summary: GUI reacts very slow
Status: RESOLVED FIXED
Alias: None
Product: akregator
Classification: Applications
Component: general (show other bugs)
Version: 1.0-beta8
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-12-03 18:13 UTC by Sebastian Stein
Modified: 2005-01-02 11:38 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sebastian Stein 2004-12-03 18:13:18 UTC
Version:           1.0-beta8 (using KDE 3.3.1, compiled sources)
Compiler:          gcc version 3.3.4
OS:                Linux (i686) release 2.4.28

For me the GUI seems to be very slow. Clicking on another Feed takes some seconds to show up the news list. Also pressing Alt+R to mark everything as read takes too long. I bet the implementation is not that good. For example in konqueror I can browse to a folder with several thousand files without having really to wait. So an optimisation is really needed.
Comment 1 George Staikos 2004-12-04 02:53:48 UTC
Did you use a package, or code from CVS?  If CVS, when did you last update?
Comment 2 Sebastian Stein 2004-12-04 09:12:14 UTC
George Staikos <staikos@kde.org> [041204 09:09]:
> ------- Additional Comments From staikos kde org  2004-12-04 02:53 -------
> Did you use a package, or code from CVS?  If CVS, when did you last update?

I used the version 1.0 beta 8. Compared to other KDE apps it is very slow.


Sebastian

Comment 3 Sebastian Stein 2004-12-04 09:22:56 UTC
Sebastian Stein <seb_stein@gmx.de> [041204 09:22]:
> ------- Additional Comments From seb_stein gmx de  2004-12-04 09:12 -------
> George Staikos <staikos kde org> [041204 09:09]:
> > ------- Additional Comments From staikos kde org  2004-12-04 02:53 -------
> > Did you use a package, or code from CVS?  If CVS, when did you last update?
> 
> I used the version 1.0 beta 8. Compared to other KDE apps it is very slow.

And I compiled from sources.


Sebastian

Comment 4 George Staikos 2004-12-04 17:42:04 UTC
Should be fixed.  Please try with CVS-HEAD or the next release.  The previous code did 2x as many mallocs and did other things inefficiently.
Comment 5 Sebastian Stein 2005-01-02 09:50:05 UTC
George Staikos <staikos@kde.org> [041205 09:32]:
> ------- You are receiving this mail because: -------
> You reported the bug, or are watching the reporter.
>         
> http://bugs.kde.org/show_bug.cgi?id=94337        
> staikos kde org changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|UNCONFIRMED                 |RESOLVED
>          Resolution|                            |FIXED
> 
> 
> 
> ------- Additional Comments From staikos kde org  2004-12-04 17:42 -------
> Should be fixed.  Please try with CVS-HEAD or the next release.  The
> previous code did 2x as many mallocs and did other things inefficiently.

Could you please upload a new package? I was not able to compile the CVS
code, maybe because I'm using KDE 3.3 libs or whatever. 

Sebastian

Comment 6 Frank Osterfeld 2005-01-02 10:02:07 UTC
Actually KDE-PIM is supposed to work with KDE 3.3 libs. Could you post the error? 
Releasing a standalone tarball is a good idea though. We should go for that after fixing some nasty bugs.
Comment 7 Sebastian Stein 2005-01-02 11:38:13 UTC
Frank Osterfeld <frank.osterfeld@kdemail.net> [050102 10:54]:
> Actually KDE-PIM is supposed to work with KDE 3.3 libs. Could you post the
> error? 

Ok, I tried some minutes ago the current code in kdepim/ module:

/usr/lib/qt/bin/moc ./feed.h -o feed.moc
/opt/kde/bin/kconfig_compiler ./akregator.kcfg ./akregatorconfig.kcfgc;
ret=$?; \
if test "$ret" != 0; then rm -f akregatorconfig.h ; exit $ret ;  fi
if /bin/sh ../../libtool --silent --tag=CXX --mode=compile g++
-DHAVE_CONFIG_H -I. -I. -I../.. -I../../akregator/src/librss -I../..
-I../../libkdepim -I/opt/kde/include -I/usr/lib/qt/include
-I/usr/X11R6/include   -DQT_THREAD_SUPPORT  -D_REENTRANT
-D_FILE_OFFSET_BITS=64  -Wnon-virtual-dtor -Wno-long-long -Wundef -ansi
-D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align -Wconversion
-Wchar-subscripts -Wall -W -Wpointer-arith -Wwrite-strings -g -O2
-Wformat-security -Wmissing-format-attribute -fno-exceptions -fno-check-new
-fno-common -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL
-DQT_NO_COMPAT -DQT_NO_TRANSLATION  -MT feed.lo -MD -MP -MF ".deps/feed.Tpo"
-c -o feed.lo feed.cpp; \
then mv -f ".deps/feed.Tpo" ".deps/feed.Plo"; else rm -f ".deps/feed.Tpo";
exit 1; fi
feed.cpp:13:41: progressmanager.h: No such file or directory
feed.cpp: In member function `void Akregator::Feed::slotSetProgress(long
   unsigned int)':
feed.cpp:356: error: `setProgress' undeclared (first use this function)
feed.cpp:356: error: (Each undeclared identifier is reported only once for
each
   function it appears in.)
feed.cpp: In member function `void Akregator::Feed::tryFetch()':
feed.cpp:369: error: no class template named `ProgressManager' in `KPIM'
feed.cpp:369: error: no class template named `ProgressManager' in `KPIM'
feed.cpp: In member function `void
   Akregator::Feed::fetchCompleted(RSS::Loader*, RSS::Document,
RSS::Status)':
feed.cpp:390: error: `setStatus' undeclared (first use this function)
feed.cpp:402: error: `setComplete' undeclared (first use this function)
make[2]: *** [feed.lo] Fehler 1

> Releasing a standalone tarball is a good idea though. We should go for
> that after fixing some nasty bugs.

Ah, the instructions of the akregator homepage need an update, because there
it is still written that akregator is in kdenonbeta. 

Sebastian