Skip to content

Conversation

@IhorNehrutsa
Copy link
Contributor

#if DEBUG_PRINT is used 7 times

image

#ifdef DEBUG_PRINT is used 3 times.

image

Make them the same #if DEBUG_PRINT.

@github-actions
Copy link

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS

@codecov
Copy link

codecov bot commented Sep 26, 2023

Codecov Report

Merging #12520 (cce0635) into master (2fcd28f) will not change coverage.
The diff coverage is n/a.

❗ Current head cce0635 differs from pull request most recent head d83c1a4. Consider uploading reports for the commit d83c1a4 to get more accurate results

@@           Coverage Diff           @@
##           master   #12520   +/-   ##
=======================================
  Coverage   98.39%   98.39%           
=======================================
  Files         158      158           
  Lines       20938    20938           
=======================================
  Hits        20601    20601           
  Misses        337      337           
Files Coverage Δ
py/emitglue.c 100.00% <ø> (ø)
py/emitnative.c 99.31% <ø> (ø)

@dpgeorge dpgeorge added the py-core Relates to py/ directory in source label Sep 29, 2023
Signed-off-by: Ihor Nehrutsa <Ihor.Nehrutsa@gmail.com>
@dpgeorge dpgeorge force-pushed the ifdef_DEBUG_PRINT_to_if_DEBUG_PRINT branch from cce0635 to d83c1a4 Compare September 29, 2023 03:05
@dpgeorge dpgeorge merged commit d83c1a4 into micropython:master Sep 29, 2023
@dpgeorge
Copy link
Member

Thank you.

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

Labels

py-core Relates to py/ directory in source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants