v 0.7.0 with runtime pin IO

Author: nerdralph

README.md

@@ -6,8 +6,5 @@ The Arduino Blink example sketch built for the Uno takes 924 bytes of flash, but
 Download [a release](https://github.com/nerdralph/ArduinoShrink/releases), and extract the files into your Arduino/libraries folder.  Select ArduinoShrink from the Sketch, Include Library menu.  This will replace several Arduino functions with smaller and faster versions.  The Arduino pin number must be known at compile time if you are using this library.
 
 ## Design and Implemenation
-I've written <a href="http://nerdralph.blogspot.com/2021/04/honey-i-shrunk-arduino-core.html">a blog post</a> discussing the implementation details of ArduinoShrink.
-
-## Planned Improvements
-Update digitalWrite() to work when the pin is not known at compile time.
+I've written <a href="http://nerdralph.blogspot.com/2021/04/honey-i-shrunk-arduino-core.html">a blog post</a> discussing the implementation details of ArduinoShrink.  Version 0.7.0 supports manipulation of pins that are not known at compile-time.  When the pin is known at compiletime, digitalWrite() will still compile to a single "sbi" or "cbi" instruction.
 

library.properties

@@ -1,5 +1,5 @@
 name=ArduinoShrink
-version=0.6.0
+version=0.7.0
 author=Ralph Doncaster
 maintainer=
 sentence=

src/wiring.c

@@ -1,6 +1,7 @@
 // (c) Ralph Doncaster 2020
 // ArduinoShrink
 
+#define ARDUINO_MAIN
 #include <Arduino.h>
 #include <util/delay.h>
 #include "as_common.h"

src/wiring_digital.c

@@ -2,32 +2,47 @@
 // thanks to Bill W for inspiration
 // github.com/WestfW/Duino-hacks/blob/master/fastdigitalIO/fastdigitalIO.h 
 // 20200709 v0.1.0 release
+// 20200410 alias _to_xxx_PGM[] arrays as __flash for runtime pin manipulation
 
-#define ARDUINO_MAIN
-#include <Arduino.h>
+#include <stdint.h>
+#include <avr/io.h>
+
+// todo: use constants.h from PicoCore
+enum {LOW, HIGH};                       // for digitalWrite()
+enum {INPUT, OUTPUT, INPUT_PULLUP};     // for pinMode()
+
+extern __flash const uint16_t port_to_output_PGM[];
+extern __flash const uint8_t digital_pin_to_port_PGM[];
+extern __flash const uint8_t digital_pin_to_bit_mask_PGM[];
 
 typedef volatile uint8_t* ioreg_p;
 
-inline ioreg_p pin_to_port(uint8_t pin)
+#define ALWAYS_INLINE __attribute__((always_inline)) inline
+
+ALWAYS_INLINE ioreg_p pin_to_port(uint8_t pin)
 {
-    return (ioreg_p) port_to_output_PGM[digital_pin_to_port_PGM[pin]];
+    // use pointer instead of array operators due to avr-gcc bug:
+    // https://www.avrfreaks.net/forum/avr-gcc-have-i-told-you-lately-i-hate-you
+    //return (ioreg_p) port_to_output_PGM[digital_pin_to_port_PGM[pin]];
+    return (ioreg_p) *(port_to_output_PGM + *(digital_pin_to_port_PGM +pin));
 }
 
-inline ioreg_p pin_to_ddr(uint8_t pin)
+ALWAYS_INLINE ioreg_p pin_to_ddr(uint8_t pin)
 {
     // DDR address is 1 less than PORT address
     return pin_to_port(pin) - 1;
 }
 
-inline ioreg_p pin_to_pinreg(uint8_t pin)
+ALWAYS_INLINE ioreg_p pin_to_pinreg(uint8_t pin)
 {
     // PIN address is 2 less than PORT address
     return pin_to_port(pin) - 2;
 }
 
-inline uint8_t pin_to_bit(uint8_t pin)
+ALWAYS_INLINE uint8_t pin_to_bit(uint8_t pin)
 {
-    return digital_pin_to_bit_mask_PGM[pin];
+    //return digital_pin_to_bit_mask_PGM[pin];
+    return *(digital_pin_to_bit_mask_PGM + pin);
 }
 
 void pinMode(uint8_t pin, uint8_t mode)