Skip to content

Add tell support for mounted files - #18298

Closed
glueckm wants to merge 1 commit into
micropython:masterfrom
glueckm:master
Closed

Add tell support for mounted files#18298
glueckm wants to merge 1 commit into
micropython:masterfrom
glueckm:master

Conversation

@glueckm

@glueckm glueckm commented Oct 19, 2025

Copy link
Copy Markdown

Summary

For one of my projected I need support for the tell function on remote files.
So I took a shot at implementing it.

@github-actions

github-actions Bot commented Oct 19, 2025

Copy link
Copy Markdown

Code size report:

  mpy-cross:    +0 +0.000% 
   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_W
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS
  qemu rv32:    +0 +0.000% VIRT_RV32

@dpgeorge dpgeorge added the tools Relates to tools/ directory in source, or other tooling label Oct 22, 2025
@dpgeorge

dpgeorge commented Jul 14, 2026

Copy link
Copy Markdown
Member

Thanks for the contribution.

It's a good feature to add, but I think it would be a lot simpler to implement it as:

def tell(self):
    return self.seek(0, 1)  # SEEK_CUR

@dpgeorge dpgeorge mentioned this pull request Aug 28, 2026
Open
@dpgeorge

dpgeorge commented Aug 28, 2026

Copy link
Copy Markdown
Member

I'll close this in favour of #19663, which includes a test.

@dpgeorge dpgeorge closed this Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

tools Relates to tools/ directory in source, or other tooling

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@glueckm @dpgeorge