Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build issue on VS 2010 #23

Closed
GoogleCodeExporter opened this issue May 28, 2015 · 6 comments
Closed

Build issue on VS 2010 #23

GoogleCodeExporter opened this issue May 28, 2015 · 6 comments

Comments

@GoogleCodeExporter
Copy link

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
Copy link
Author

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
Copy link
Author

Thank you for quick response

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

@GoogleCodeExporter
Copy link
Author

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
Copy link
Author

[deleted comment]

@GoogleCodeExporter
Copy link
Author

Fixed in https://pdfium.googlesource.com/pdfium/+/6256a14.

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

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Thanks!

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant