IntelHex

Patch: implementation of IntelHex.loadbin

Bug #110310 reported by Alex Mueller
2
Affects Status Importance Assigned to Milestone
IntelHex
Fix Released
Medium
Alexander Belchenko
IntelHex 0.9.0

Bug Description

This funcionality was missing. Hopefully my patch addresses this issue (no unit test added).

Related branches

lp:intelhex/0.9
Revision history for this message
Alex Mueller (xelarellum) wrote :
Revision history for this message
Alexander Belchenko (bialix) wrote :

It's indeed missing.

Changed in intelhex:
importance: Undecided → Medium
status: Unconfirmed → Confirmed
Revision history for this message
Alexander Belchenko (bialix) wrote :

I think that updating of self._offset is not needed when binary file loading, because it's not used in loadbin.
self._offset used only for read hex files.

Alex: Do you want to write a unit test for this method? It's not hard at all, really.

Revision history for this message
Alex Mueller (xelarellum) wrote :

Hopefully I will the time for such next week.

Revision history for this message
Alexander Belchenko (bialix) wrote :

implementation + tests added to trunk

Changed in intelhex:
assignee: nobody → bialix
status: Confirmed → Fix Committed
Alexander Belchenko (bialix)
Changed in intelhex:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.