Skip to content

Conversation

@shakna-israel
Copy link
Owner

No description provided.

@shakna-israel shakna-israel added this to the 1.7.0 milestone Jan 23, 2018
@shakna-israel
Copy link
Owner Author

Left and right bitshifts have been added, and should be working. That covers #61.

@shakna-israel
Copy link
Owner Author

Since bit32 got backported to Lua 5.1, I've removed the bitshift stuff, and just relying on the inbuilt library.

Base64 encode implemented and working. Now just the decode side of things.

@shakna-israel
Copy link
Owner Author

f.lua is > 1000 LOC. Time to change styles a bit. To f.x instead of x, return x

@shakna-israel
Copy link
Owner Author

Failure explanation: bit32 not available under Lua 5.1 or Luajit....

@shakna-israel
Copy link
Owner Author

Successfully patched the necessary (for base64) bit32 operations for Lua 5.1, Luajit.

@codecov-io
Copy link

codecov-io commented Mar 20, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@de4a068). Click here to learn what that means.
The diff coverage is 90.95%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #66   +/-   ##
=========================================
  Coverage          ?   81.85%           
=========================================
  Files             ?        1           
  Lines             ?      518           
  Branches          ?        0           
=========================================
  Hits              ?      424           
  Misses            ?       94           
  Partials          ?        0
Impacted Files Coverage Δ
f.lua 81.85% <90.95%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de4a068...2eaa876. Read the comment docs.

@shakna-israel
Copy link
Owner Author

Progress has slowed - I am on bedrest, told to avoid stress. (Hole in my heart).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants