GSoC '21 KMyMoney: Removed all usage of webpricequote in equity price update dialog source.

[Thomas Baumgart]

On Dienstag, 3. August 2021 20:47:12 CEST Suraj Kumar Mahto via Kde-finance-apps wrote:

> On Tue, 3 Aug, 2021, 1:36 am Thomas Baumgart via Kde-finance-apps, <
> kde-finance-apps at kde.org> wrote:
> 
> > It might be better (performance wise) to add those includes to the
> > kequitypriceupdatedlg.cpp file
> > and maybe others that also include the kequitypriceupdatedlg.h file.
> >
> > The forward declaration usually uses less resources while compiling than
> > including the full
> > class definition. Since the header file only uses references or pointers
> > to those objects,
> > knowing that the classes exist is enough for the compiler to deal with the
> > class definition
> > of KEquityPriceDlg.
> >
> 
> Sorry for the late response. Using forward declaration of those classes
> created many conflicts after removing the include statement of
> webpricequote.h in kequitypriceupdatedlg.cpp
> Using header files directly however, wasn't behaving the same and it
> compiled without errors.
> 
> BTW, My last week's blog that I published yesterday :
> 
> https://suraj-sloth.github.io/2021/08/02/gsoc21-week8.html

I saw that. It got posted on planet.kde.org

> Also, I wanted to ask if we can have a virtual meet this weekend if mentors
> are free to discuss about the project status and work left.

We can do that. Let me check my availability and I will get back here.


-- 

Regards

Thomas Baumgart

https://www.signal.org/       Signal, the better WhatsApp
-------------------------------------------------------------
Windows is not the answer. It's the question. The answer is no!
-------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 868 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-finance-apps/attachments/20210805/d8c03ca7/attachment.sig>