Skip to content

Conversation

@avsm
Copy link
Member

@avsm avsm commented Oct 12, 2013

No description provided.

@djs55
Copy link

djs55 commented Oct 12, 2013

The readme improvements are especially useful! I'm always trying to
remember exactly what is being generated.

Sent from my iPhone

On 12 Oct 2013, at 23:04, Anil Madhavapeddy notifications@github.com
wrote:


You can merge this Pull Request by running

git pull https://github.com/avsm/ocaml-cstruct master

Or view, comment on, or merge it at:

mirage#11
Commit Summary

  • Add experimental Travis setup
  • correct add-apt-repository
  • simplify scripts, add badge
  • Make build verbose
  • add test script
  • Merge branch 'travis'
  • Sync CHANGES
  • travis: install native compilers
  • sync oasis autogen
  • install time package in Travis
  • improve the Travis test matrix with the latest PPAs
  • Check and raise error in case of negative offsets for blits
  • travis: install time
  • a candidate fix for add constant constraints in enums mirage/ocaml-cstruct#3
  • better fix for merge- conflict around getn/getz helper functions mirage/ocaml-cstruct#4 so that parsing outputs the
    right integer literals
  • test: fix enum test compilation
  • tests: restore pcap printing
  • tests: make basic test compile again with new api
  • sync CHANGES
  • Correctly preserve the sequence after a constant constructor is set
    during a cenum definition.
  • Do not repeat the sizeof_<field> binding for every get/set field
    (should be no externally observable change).
  • Do not repeat the sizeof_<field> signature for every get/set field.
  • update README with more information about the generated functions
  • Merge pull request Documentation and bounds checking improvements #18 from avsm/maybe_fix_cstruct_32bit_overflow

File Changes

Patch Links:

avsm added a commit that referenced this pull request Oct 12, 2013
Fix cstruct overflow, correct sequencing in cenum, add docs
@avsm avsm merged commit 554f6a8 into ocaml-attic:master Oct 12, 2013
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.

2 participants