SD get pointer block

[VonSzarvas]

Add the SD pointer block and set to "get".
Notice the block is not formed to allow the result variable to be attached.
Click code and get white screen. Click compile and error or hang.

[AndyLindsay]

item = SD file (get) pointer emits item = fp; It should instead emit item = ftell(fp); In propeller C, this value is negative. It might be better to use a larger routine that returns a pre-digested value that the user can apply in SD file (set) pointer.

Also, SD file (set) pointer = item emits fp = item. It should instead emit fseek(fp, item, SEEK_CUR);

[zfi]

The block geometry of the two different states of this block are incompatible. When in default the 'set' mode, the block acts like a program element. It has top and bottom connectors and accepts an input value used to set the pointer value.

When the block is in the 'get' mode, the geometry changes completely from a program element to an input element. The top and bottom connectors are removed as is the connector for an input value.

These two modes of operation create a number of issues when the block is attached to another block, either as an input value or as a program element, and the type of the block is changed to the alternate value. For example, when the type is 'set' and the block is attached inline with other program elements, changing the block type to 'get' causes a couple of problems.

• The block attempts to remove the top and bottom connectors and fails to do so because the connectors are attached to other blocks.
• The block attempts to remove the input connector and fails if there is a value or variable block attached here.

Moving from a 'get' type to a 'set' type creates similar issues in that the the block no longer acts as an input so the connection to the input of the attached block fails. Similarly, the block changes to have programatic elements with a top and bottom connection but not attached to anything.

The two states literally are completely different things. It seems to me that the 'get' element needs to be a new, separate block.

[zfi]

@AndyLindsay, I am not quite getting the exact purpose of the get pointer block mode. If the block is supposed to seek, wouldn't we need an input value to say how far to seek from the current position? As it is now, it just returns the FILE *.

[AndyLindsay]

@zfi, I'm suggesting that the SD file (set) pointer = item blocks emit fseek(fp, item, SEEK_CUR); That will move the pointer to the desired position in the file. In the example, I'm using the item variable, and assuming it stores an int, but it could also be a value block.

For SD file (get) pointer, I'm suggesting that it call ftell(fp), which returns the pointer's offset in the file. So, in item = SD file (get) pointer, the item variable would receive the result of the ftell(fp) call.

[AndyLindsay]

Regarding the SD file (set/get) pointer, I do agree that it would make sense to have two separate blocks: SD file set pointer = and = SD file get pointer. This would be consistent with the most basic operations in Pin states, where we have make PIN (0) = [high] and = check PIN [0]. Their functions are inverses of each other, and so they separate blocks, one with the input shape, and the other with the output shape. The SD blocks have not received much use yet, so I would not expect changing them from one to two blocks would have minimal impact.

[AndyLindsay]

@zfi, P.S. Regarding what I said about the SD file (set) pointer = item blocks emitting fseek(fp, item, SEEK_CUR), are you suggesting to use fseek(fp, item, SEEK_SET) instead? If so, I agree. Sorry, got my constants mixed up.

[zfi]

The single block will continue to offer file pointer 'get' and 'set' functionality until we refactor this into two separate blocks. The 'set' mode now uses fseek() with an anchor reference to the start of the file (SEEK_SET). The block, when configured initially to the 'set' mode will generate console error messages if the mode is changed to 'get' while the block is attached to any other block. It is this behavior that is driving the requirement for separate blocks for 'set' and 'get'.