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

Issues discovered on Mac OS X #131

Open
smpeters opened this issue Mar 29, 2019 · 2 comments
Open

Issues discovered on Mac OS X #131

smpeters opened this issue Mar 29, 2019 · 2 comments

Comments

@smpeters
Copy link
Contributor

I was able to compile the most recent Github version and everything worked fine. That said, there were a few issues.

  • Makefile.PL said Warning: prerequisite JSON::MaybeXS 0 not found. . If everything compiles and all the tests pass it is either not a real prerequisite or there is no test coverage for code where JSON::MaybeXS is required.
  • NYTProf.xs:1220:1: warning: unused function 'get_str_id' [-Wunused-function]
  • several vararg warnings are displayed similar to
NYTProf.xs:3999:20: warning: passing an object that undergoes default argument promotion to 'va_start' has undefined behavior
      [-Wvarargs]
    va_start(args, tag);
@jkeenan
Copy link
Collaborator

jkeenan commented Apr 6, 2021

I was able to compile the most recent Github version and everything worked fine. That said, there were a few issues.

* `Makefile.PL` said `Warning: prerequisite JSON::MaybeXS 0 not found.` . If everything compiles and all the tests pass it is either not a real prerequisite or there is no test coverage for code where `JSON::MaybeXS` is required.

[snip]

NYTProf.xs:3999:20: warning: passing an object that undergoes default argument promotion to 'va_start' has undefined behavior
      [-Wvarargs]
    va_start(args, tag);
$ ack 'JSON::MaybeXS' bin lib t
bin/nytprofhtml
70:my $has_json = eval { require JSON::MaybeXS; JSON::MaybeXS->import(); 1 }
71:    or warn "Can't load JSON::MaybeXS module - HTML visualizations skipped ($@)\n";
873:        print $fh q{<br/>(Can't create visual treemap of subroutine exclusive times without the <a href="http://metacpan.org/release/JSON-MaybeXS/">JSON::MaybeXS</a> module.)<br/>};

@jkeenan
Copy link
Collaborator

jkeenan commented Oct 6, 2023

I was able to compile the most recent Github version and everything worked fine. That said, there were a few issues.

* `Makefile.PL` said `Warning: prerequisite JSON::MaybeXS 0 not found.` . If everything compiles and all the tests pass it is either not a real prerequisite or there is no test coverage for code where `JSON::MaybeXS` is required.

Previously responded to.

* `NYTProf.xs:1220:1: warning: unused function 'get_str_id' [-Wunused-function]`

* several vararg warnings are displayed similar to
NYTProf.xs:3999:20: warning: passing an object that undergoes default argument promotion to 'va_start' has undefined behavior
      [-Wvarargs]
    va_start(args, tag);

I don't have access to Mac OS X. Can you tell me what C-compiler you were using and what version thereof? Thanks.

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

No branches or pull requests

2 participants