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.

Sign up for GitHub

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

Jump to bottom

Basic compilation fixes #1

Merged
merged 7 commits into from
Oct 12, 2014
Merged

Basic compilation fixes #1

merged 7 commits into from
Oct 12, 2014

Conversation

webmaster128
Copy link
Contributor

No description provided.

@webmaster128
Add year and name to MIT license
7dc43ef
@webmaster128
Add ~ files to git ignore list
db14394
@webmaster128
Remove developer specific include and linker paths
cc3f722
@webmaster128
Add missing include for size_t
b01a446 
Fixes compile error:
[...]
g++ -c  -g TokenArray.cpp
In file included from TokenArray.cpp:1:0:
TokenArray.h:20:4: error: ‘size_t’ does not name a type
    size_t size() {return m_tokens.size();}
    ^
TokenArray.h:41:4: error: ‘size_t’ does not name a type
    size_t m_index;
    ^
@webmaster128
Fix object order for linker
38d3eab 
otherwise g++ drops PCRE symbols
@webmaster128
Remove whitespace in makefile
77ec6dd
@webmaster128
Fix typo: "Formual" -> "Formula"
370951a
@lishen2 lishen2 merged commit 370951a into lishen2:master Oct 12, 2014
@lishen2
Copy link
Owner

lishen2 commented Oct 12, 2014

Merged, not as hard as I thought.

Sent from my iPhone

On 2014年10月7日, at 02:07, Simon Warta [email protected] wrote:

You can merge this Pull Request by running

git pull https://github.com/webmaster128/Excel_formula_parser_cpp master
Or view, comment on, or merge it at:

#1

Commit Summary

Add year and name to MIT license
Add ~ files to git ignore list
Remove developer specific include and linker paths
Add missing include for size_t
Fix object order for linker
Remove whitespace in makefile
Fix typo: "Formual" -> "Formula"
File Changes

A .gitignore (2)
M LICENSE (2)
M TokenArray.h (1)
M main.cpp (2)
M makefile (11)
Patch Links:

https://github.com/lishen2/Excel_formula_parser_cpp/pull/1.patch
https://github.com/lishen2/Excel_formula_parser_cpp/pull/1.diff
—
Reply to this email directly or view it on GitHub.

@webmaster128
Copy link
Contributor Author

1 Click on Github ;)

Mobil gesendet.

Am 12.10.2014 um 15:15 schrieb lishen [email protected]:

Merged, not as hard as I thought.

Sent from my iPhone

On 2014年10月7日, at 02:07, Simon Warta [email protected] wrote:

You can merge this Pull Request by running

git pull https://github.com/webmaster128/Excel_formula_parser_cpp master
Or view, comment on, or merge it at:

#1

Commit Summary

Add year and name to MIT license
Add ~ files to git ignore list
Remove developer specific include and linker paths
Add missing include for size_t
Fix object order for linker
Remove whitespace in makefile
Fix typo: "Formual" -> "Formula"
File Changes

A .gitignore (2)
M LICENSE (2)
M TokenArray.h (1)
M main.cpp (2)
M makefile (11)
Patch Links:

https://github.com/lishen2/Excel_formula_parser_cpp/pull/1.patch
https://github.com/lishen2/Excel_formula_parser_cpp/pull/1.diff
—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@webmaster128 @lishen2