Bug 205

Summary:	.align directive
Product:	RMAC	Reporter:	ggn <ggnkua>
Component:	Core	Assignee:	Shamus Hammons <jlhamm>
Status:	RESOLVED FIXED
Severity:	enhancement	CC:	jlhamm
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	All
Attachments:	Bastian's original patch (for posterity) My patch (including doc updates and some bits here and there) Second try (apparently some people define "align" as a macro) No idea what happened to the previous one. Absolutely none

Description ggn 2022-07-23 12:12:28 CDT

Bastian shared a patch for arbitrary alignment here: https://atariage.com/forums/topic/234634-rmacrln/?do=findComment&comment=5079657

I merely tested it under a few configurations and added proper 56001 handling. Looks fine to me!

Comment 1 ggn 2022-07-23 12:12:53 CDT

Created attachment 187 [details]
Bastian's original patch (for posterity)

Comment 2 ggn 2022-07-23 12:14:05 CDT

Created attachment 188 [details]
My patch (including doc updates and some bits here and there)

Comment 3 ggn 2022-07-25 07:45:00 CDT

Created attachment 192 [details]
Second try (apparently some people define "align" as a macro)

Comment 4 Shamus Hammons 2022-08-15 21:14:56 CDT

Well, the patch doesn't cleanly apply from HEAD (currently at db581e2d9efe477db3a70fb8e947f3d009545d09).  And of course, git doesn't tell you where in the patch it fails.  :-(

Comment 5 ggn 2022-08-16 01:27:59 CDT

Created attachment 198 [details]
No idea what happened to the previous one. Absolutely none

Comment 6 ggn 2022-08-16 01:28:43 CDT

Okay, I remade the patch because I've 0 ideas what happened there. This should apply cleanly...

Comment 7 Shamus Hammons 2022-08-16 12:23:22 CDT

All right, we have a winner!  :-D