-
Notifications
You must be signed in to change notification settings - Fork 807
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
Business objects cpp #1985
Draft
christopherlam
wants to merge
17
commits into
Gnucash:stable
Choose a base branch
from
christopherlam:business-objects-cpp
base: stable
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Business objects cpp #1985
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
93719bb
[*.h] add enumtypes = 0 as used by gncInvoice.cpp and gncEntry.cpp
christopherlam 35751aa
[gncOwner.cpp] convert to c++
christopherlam dddb267
[gncInvoice.cpp] convert to c++
christopherlam 26a8248
[gncCustomer.cpp] convert to c++
christopherlam 047afc7
[gncVendor.cpp] convert to c++
christopherlam d00e36d
[gncEmployee.cpp] convert to c++
christopherlam fa06c13
[gncEntry.cpp] convert to c++
christopherlam ff88876
[gncAddress.cpp] convert to c++
christopherlam 4beff7f
[gncBillTerm.cpp] convert to c++
christopherlam 3bb39bf
[gncTaxTable.cpp] convert to c++
christopherlam 0d5883d
Revert "[*.h] add enumtypes = 0 as used by gncInvoice.cpp and gncEntr…
christopherlam b4f3f83
fixup! [gncEntry.cpp] amend to avoid returning invalid discount values
christopherlam 61a9ede
fixup! [gncInvoice.cpp] amend to avoid using invalid types
christopherlam 03e35eb
[gncJob.cpp] convert to c++
christopherlam b6caa92
[gncOrder.cpp] convert to c++
christopherlam c822d84
[gnc-uri-utils.cpp] convert to c++
christopherlam 0d64531
fixup! [gncBillTerm.cpp] don't return invalid enum
christopherlam File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -77,6 +77,7 @@ typedef struct _gncTaxTableClass GncTaxTableClass; | |
*/ | ||
typedef enum | ||
{ | ||
GNC_AMT_TYPE_DISABLED = 0, /** no tax */ | ||
GNC_AMT_TYPE_VALUE = 1, /**< tax is a number */ | ||
GNC_AMT_TYPE_PERCENT /**< tax is a percentage */ | ||
} GncAmountType; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Those are remarkably useless comments. |
||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As long as you increment by 1 you need only assign the first one, and 0 is the default so you can just add the 0 enum and skip the initializer, e.g.