> 530 + virtual ~PreviewPreCacher();
> 551 + virtual void OnPreCacheComplete();
>
> Could we make those pure virtual fucntions?
>
> Or if not could we just do:
> 530 + virtual ~PreviewPreCacher() {}
> 551 + virtual void OnPreCacheComplete() {}
>
Can't be pure virtual because we need to instantiate a PreviewPreCacher.
Don't see any benefit in making them inline.
>
> 347 + if (index < 0 || index >=(int)GetResultCount())
>
> Does this need to be casted to an int?
>
Yes. GetResultCount returns an unsigned int. Comes from the dee model.
> 530 + virtual ~PreviewPreCach er(); ete(); ete() {}
> 551 + virtual void OnPreCacheCompl
>
> Could we make those pure virtual fucntions?
>
> Or if not could we just do:
> 530 + virtual ~PreviewPreCacher() {}
> 551 + virtual void OnPreCacheCompl
>
Can't be pure virtual because we need to instantiate a PreviewPreCacher.
Don't see any benefit in making them inline.
> GetResultCount( ))
> 347 + if (index < 0 || index >=(int)
>
> Does this need to be casted to an int?
>
Yes. GetResultCount returns an unsigned int. Comes from the dee model.
> info_max_ width = MAX(details_width - nAreaWidth( ) - style.GetSpaceB etweenIconAndDe tails() , 0); cr.width* 0.1)); >SetMaximumWidt h(MAX(0, GetGeometry().width - geo.height - urationWidth( ) - layout_spacing*2));
> 758 + int top_social_
> style.GetAppIco
> 772 + int max_width = MAX(0, geo_cr.width - 2*(geo_
> 773 + int max_height = MAX(0, (geo_cr.height - TAIL_HEIGHT) -
> 2*((geo_cr.height - TAIL_HEIGHT)*0.1));
> 786 + title_-
> style.GetMusicD
>
> Lets use std::max() here.
Will do.