h0wl / pdfium

Automatically exported from code.google.com/p/pdfium
1 stars 1 forks source link

Build issue on VS 2010 #23

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hello

When I build project using VS 2010, I have an issue: 
http://pastebin.com/gWQ4gWRR.

I discover that it comes from [cceb554] commit and connected with FT_Vector 
font_offset initialization.

It works right when struct inits that way:

FT_Vector               font_offset = {0, 0};

I attached path for this issue.
Sorry for my English :)

Original issue reported on code.google.com by anatol1...@gmail.com on 9 Jul 2014 at 12:35

Attachments:

GoogleCodeExporter commented 9 years ago
I met the same issue, and here is the patch I used, 

refer to 
http://stackoverflow.com/questions/9903582/error-c2275-illegal-use-of-this-type-
as-an-expression  
Your code cannot be compiled as C++ code.  You'll have to follow 'C' rulez, a 
local variable declaration must appear before any statements in a function 
body.  

Original comment by roc...@gmail.com on 10 Jul 2014 at 5:40

Attachments:

GoogleCodeExporter commented 9 years ago
Thank you for quick response

Original comment by anatol1...@gmail.com on 10 Jul 2014 at 6:43

GoogleCodeExporter commented 9 years ago
I reported this issue in the corresponding codereview with a patch:

https://codereview.chromium.org/354413002/#msg4

The response was that VS 2010 is not supported. The rest of the code builds 
fine with the compiler of VS 2010, so could you make an exception and merge 
this trivial fix? 

Original comment by m.dopp...@gmail.com on 17 Jul 2014 at 6:04

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Fixed in https://pdfium.googlesource.com/pdfium/+/6256a14.

Original comment by jun_f...@foxitsoftware.com on 18 Jul 2014 at 4:38

GoogleCodeExporter commented 9 years ago
Thanks!

Original comment by m.dopp...@gmail.com on 21 Jul 2014 at 7:53