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

const support #2

Open
wants to merge 39 commits into
base: master
Choose a base branch
from
Open

const support #2

wants to merge 39 commits into from

Conversation

rockkoca
Copy link
Owner

@rockkoca rockkoca commented Mar 1, 2017

No description provided.

rockkoca and others added 26 commits February 27, 2017 23:04
* const test

* add method const modifier support by appending {query} to _type.
Passed unit tests
# Additional function : --version
- add --version arguments in command line
- I think that modified version will be 0.3

# Bug : What is the problem
- we have the error when we use enum without name.
```
enum {
    AAAAAAA,
    BBBBBBB,
};
```

- Error Message
```
[1133] WARN-enum: nameless enum ['enum', '{', 'AAAAAAA', ',', 'BBBBBBB', ',', '}']
Traceback (most recent call last):
  File "/Users/cheoljoo/.local/bin/hpp2plantuml", line 11, in <module>
    load_entry_point('hpp2plantuml==0.2', 'console_scripts', 'hpp2plantuml')()
  File "/Users/cheoljoo/.local/lib/python3.6/site-packages/hpp2plantuml-0.2-py3.6.egg/hpp2plantuml/hpp2plantuml.py", line 1110, in main
    CreatePlantUMLFile(args.input_files, args.output_file)
  File "/Users/cheoljoo/.local/lib/python3.6/site-packages/hpp2plantuml-0.2-py3.6.egg/hpp2plantuml/hpp2plantuml.py", line 1080, in CreatePlantUMLFile
    diag.create_from_file_list(list(set(expand_file_list(file_list_c))))
  File "/Users/cheoljoo/.local/lib/python3.6/site-packages/hpp2plantuml-0.2-py3.6.egg/hpp2plantuml/hpp2plantuml.py", line 768, in create_from_file_list
    flag_build_lists=True, flag_reset=True)
  File "/Users/cheoljoo/.local/lib/python3.6/site-packages/hpp2plantuml-0.2-py3.6.egg/hpp2plantuml/hpp2plantuml.py", line 747, in _build_helper
    self.parse_objects(single_input, build_from_single)
  File "/Users/cheoljoo/.local/lib/python3.6/site-packages/hpp2plantuml-0.2-py3.6.egg/hpp2plantuml/hpp2plantuml.py", line 860, in parse_objects
    self._objects.append(container_handler(obj))
  File "/Users/cheoljoo/.local/lib/python3.6/site-packages/hpp2plantuml-0.2-py3.6.egg/hpp2plantuml/hpp2plantuml.py", line 32, in <lambda>
    ['enums', lambda objs: objs, lambda obj: Enum(obj)]
  File "/Users/cheoljoo/.local/lib/python3.6/site-packages/hpp2plantuml-0.2-py3.6.egg/hpp2plantuml/hpp2plantuml.py", line 486, in __init__
    super().__init__('enum', header_enum['name'])
KeyError: 'name'
```

- Changed Code
	- This error generates when it is not defined the key of dictionary.
		- So define "empty" name as a default value when enum does not have the name.
	- super().__init__('enum', header_enum['name']) -> super().__init__('enum', header_enum.get('name',empty))
	- output
```
@startuml
enum empty {
	AAAAAAA
	BBBBBBB
}

@enduml
```
Add test for anonymous enum
Fix bug with multiline template parameters
Fix bug with class enum objects (not supported but should not crash)
- Merge dependency relationship
- Support complex parameter types (containers)
- Fix naming convention for aggregation (parent/child)
- Determine aggregation/composition (raw pointer->aggregation)
- Fix typos
- Fix sphinx build issues
- Integrate recursive file crawler (pull-request #10)
- Render structs as classes
- Tentative support for anonymous structs/unions
- Fix handling of nested classes
- Bump version number
- Integrate recursive file crawler (pull-request #10)
- Render structs as classes
- Tentative support for anonymous structs/unions
- Fix handling of nested classes
- Bump version number
@thibaultmarin thibaultmarin force-pushed the master branch 6 times, most recently from b30bad2 to 0f72d6e Compare May 21, 2024 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants