<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2015-03-06 20:05 GMT+01:00 Friedrich W. H. Kossebau <span dir="ltr"><<a href="mailto:kossebau@kde.org" target="_blank">kossebau@kde.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Am Freitag, 6. März 2015, 08:07:07 schrieb Elvis Stansvik:<br>
<span class="">> 2015-03-06 1:16 GMT+01:00 Friedrich W. H. Kossebau <<a href="mailto:kossebau@kde.org">kossebau@kde.org</a>>:<br>
> > Hi,<br>
> ><br>
> > so I spent quite some time scanning the sources, below is what I found as<br>
> > TODO.<br>
> ><br>
> > So the idea still is: put 3rd-party code into a subdir "3rdparty" and put<br>
> > generated and committed code into a subdir "generated", either locally or<br>
> > globally. This allows automatted handling of such code files in<br>
> > scripts/greps<br>
> > etc, and also helps understanding code when looking for the big picture.<br>
> ><br>
> > I propose to apply the changes listed below in 2.9, I do not see a risk in<br>
> > breaking something. Only the cmake files should be affected, and perhaps<br>
> > some<br>
> > includes. The more things are consistent in 2.9 with master/future<br>
> > frameworks<br>
> > branch, the better.<br>
> ><br>
> > Please also tell what generated files or 3rd-party code is missing from<br>
> > the<br>
> > list below<br>
><br>
> I did just a quick search, but it seems you've been thorough.<br>
<br>
</span>I tried at least, but the massive amount of files is hard to scan quickly or<br>
by some terms tried with "grep", so anything but sure I got them all.<br>
<span class=""><br>
> I wonder about the two files:<br>
><br>
>   krita/plugins/formats/jpeg/iccjpeg.(h|c)<br>
><br>
> They are 3rdparty code afaics, but not in their own directory. I guess the<br>
> 3rdparty README in that directory pertains to them?<br>
<br>
</span>No, that README got committed<br>
78e5ce41afb832e088850fe76461d0ccd76ef665<br>
some years after those two files<br>
0fe9b9fca98af9221cfa16d915891b28ee841987<br>
<br>
Not really sure why that commit added the README, perhaps some code snippets<br>
used in the manually written files of the same commit are in the license<br>
mentioned in the README... no idea :/<br></blockquote><div><br></div><div>Ah. Topic for a later investigation then.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> Maybe they should be given their own directory?<br>
<br>
</span>Yes, good find, at least iccjpeg.(h|c). Will move to<br>
krita/plugins/formats/jpeg/3rdparty/lcms<br></blockquote><div><br></div><div>Great.</div><div><br></div><div>Elvis</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
Cheers<br>
Friedrich<br>
_______________________________________________<br>
calligra-devel mailing list<br>
<a href="mailto:calligra-devel@kde.org">calligra-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/calligra-devel" target="_blank">https://mail.kde.org/mailman/listinfo/calligra-devel</a><br>
</div></div></blockquote></div><br></div></div>