apol added inline comments.

INLINE COMMENTS

> FlatpakBackend.cpp:509
>          FlatpakRemote *remote = FLATPAK_REMOTE(g_ptr_array_index(remotes, 
> i));
> +        fileTimestamp = flatpak_remote_get_appstream_timestamp(remote, 
> nullptr);
>  

Can you declare and set in the same line? No need to set it to null first.

REPOSITORY
  R134 Discover Software Store

BRANCH
  appstream-refreshing

REVISION DETAIL
  https://phabricator.kde.org/D7177

To: jgrulich, apol
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart, lukas

Reply via email to