|
2024-12-13
| ||
| 10:20 | • Closed ticket [c4f365470e]: Size modifiers j, q, z, t not implemented plus 5 other changes artifact: 3d6ab30d39 user: jan.nijtmans | |
| 10:18 | Fix [c4f365470e]: Size modifiers j, q, z, t not implemented. Update changes.md check-in: 6c1255aa69 user: jan.nijtmans tags: trunk, main | |
| 09:59 | Fix [c4f365470e]: Size modifiers j, q, z, t not implemented check-in: 5dba87ea38 user: jan.nijtmans tags: core-8-branch | |
|
2024-12-12
| ||
| 14:16 | • Pending ticket [c4f365470e]: Size modifiers j, q, z, t not implemented plus 6 other changes artifact: f80d50ff0b user: jan.nijtmans | |
| 14:15 | Proposed fix for [c4f365470e]: Size modifiers j, q, z, t not implemented check-in: deae6f2c25 user: jan.nijtmans tags: bug-c4f365470e | |
| 10:48 | • Ticket [c4f365470e] Size modifiers j, q, z, t not implemented status still Open with 3 other changes artifact: f7b086dd81 user: apnadkarni | |
| 10:20 | • Ticket [c4f365470e]: 3 changes artifact: 0d9ab03e8e user: apnadkarni | |
| 09:42 | • Ticket [c4f365470e]: 3 changes artifact: a90af7a10e user: jan.nijtmans | |
| 07:58 | • New ticket [c4f365470e]. artifact: 5d9b00bcb2 user: apnadkarni | |
| Ticket UUID: | c4f365470e1e9a4a713a19f0c2275b0a4620cce0 | |||
| Title: | Size modifiers j, q, z, t not implemented | |||
| Type: | Bug | Version: | 9.0 | |
| Submitter: | apnadkarni | Created on: | 2024-12-12 07:58:38 | |
| Subsystem: | 18. Commands M-Z | Assigned To: | jan.nijtmans | |
| Priority: | 5 Medium | Severity: | Minor | |
| Status: | Closed | Last Modified: | 2024-12-13 10:20:11 | |
| Resolution: | Fixed | Closed By: | jan.nijtmans | |
| Closed on: | 2024-12-13 10:20:11 | |||
| Description: |
TIP 697 as well as the manpage scan.n document the size modifiers j, q, z, t. However, this does not seem to be actually implemented? Is that an oversight or intentionally omitted with docs not updated? | |||
| User Comments: |
jan.nijtmans added on 2024-12-13 10:20:11:
Fixed [6c1255aa6906a748|here] jan.nijtmans added on 2024-12-12 14:16:25: [deae6f2c251ba097|this] should work better. Good catch! apnadkarni added on 2024-12-12 10:48:57: > Those 4 are not useful for "scan", only for "format". BTW I do think they are useful for scan as well as since using h or l instead would require a pointerSize check first. apnadkarni added on 2024-12-12 10:20:41: Also, the TIP and the manpage say the `L` modifiers behaves like `ll` (consistent with format and different from 8.6). However, it appears it still behaves like `l` as in 8.6. Given it has been TIP'ed and documented in the manpage, my preference is for the implementation to be fixed, particularly if as you say implementation is not too hard. Your TIP-697 branch seems to already have the code implemented! And as far as I can tell was merged into main. So no idea what happened! jan.nijtmans added on 2024-12-12 09:42:44: Those 4 are not useful for "scan", only for "format". I see now that in the TIP I mentioned to implement them for "scan" as well. Why???? I don't remember. Maybe I planned to do that, but never did. So, should we add it (implementation is easy, actually)? | |||