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

Outputing the function counter to XML. #223

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

giulianobelinassi
Copy link
Contributor

Now, globalFunctions tag appears in the header of the xml output file.

@rodrigosiqueira
Copy link
Member

  1. Change globalFunctions name to totalGlobalFunctions
  2. When we execute kuniri with O1 flag it optimizes the final output to use a reduced version of the elements name. totalGlobalFunctions could be reduced to tgf and count to c when -O1 is passed to the CLI.
    Thanks :)

giulianobelinassi and others added 2 commits May 24, 2017 14:02
Now, globalFunctions tag appears in the header of the xml output file.
For now, globalFunctions = totalGlobalFunctions, and has 'tfg' as short label.

Signed-off-by: Lucas Kanashiro <[email protected]>
Signed-off-by: Giuliano Belinassi <[email protected]>
After changing globalFunctions to totalGlobalFunctions some tests broke.

Signed-off-by: Lucas Kanashiro <[email protected]>
Signed-off-by: Giuliano Belinassi <[email protected]>
@rodrigosiqueira
Copy link
Member

Almost done...
I just miss some tests similar the ones find in spec/language/ruby/global_test/. Add rspec tests with multiple variation of this parameter and other similar with tests in spec/language/ruby/global_test/

Signed-off-by: Bruno Sesso <[email protected]>
Signed-off-by: Giuliano Belinassi <[email protected]>
@giulianobelinassi
Copy link
Contributor Author

Done. Check spec/language/ruby/global_test/metrics

@rodrigosiqueira
Copy link
Member

Almost done, can you just rebase your branch with dev and fix the conflicts?

giulianobelinassi and others added 2 commits June 9, 2017 15:34
Signed-off-by: giulianobelinassi <[email protected]>
Signed-off-by: Lucas Kanashiro <[email protected]>
Signed-off-by: Bruno Sesso <[email protected]>
@giulianobelinassi
Copy link
Contributor Author

giulianobelinassi commented Jun 21, 2017

We do not know why the build is failing, but all tests succeed in my machine. @rodrigosiqueira Could you run it locally?

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