Jean-Philippe Barette-LaPierre wrote:
> 
> I just commited the structural changes. You can change the VC++ files now.

Jean

Thanks a lot for this work.

Below there are some comments from me.

0. curlpp .cpp files are missing (CurlHandle.cpp, Easy.cpp etc.)
1. Copyright notice is not updated to current year.
2. Some headers have include guard beginning with CURLPP and some don't have 
this prefix. I think all should have.
3. Some headers don't have include guard.
4. Some .inl files don't have include guard.
5. NonCopyable is in both curlpp and utilspp. Shouldn't we have only one in 
utilspp?

If you agree with above I could make these changes.

Regards
Piotr Dobrogost
_______________________________________________
cURLpp mailing list
[email protected]
http://www.rrette.com/mailman/listinfo/curlpp

Reply via email to