by Blzut3 » Sun Mar 18, 2012 6:59 pm
I guess I should comment on this. The reason I haven't added it yet is I'm not sure this is the proper way to go about things. Dynamic strings are not supposed to be persistent in any way and I think the only reason this patch works is because of DrawString's caching. It isn't safe to assume that SBarInfo will pick up on the change to the ACS variable. For example draw a string on the normal statusbar and start with the full screen. Set the variable to a dynamic string and then switch to the normal status bar. SBarInfo will try to fetch the string on the tick screenblocks changes, does your patch account for that?
I think the proper solution would be to add support for drawing character arrays with SBarInfo.
I guess I should comment on this. The reason I haven't added it yet is I'm not sure this is the proper way to go about things. Dynamic strings are not supposed to be persistent in any way and I think the only reason this patch works is because of DrawString's caching. It isn't safe to assume that SBarInfo will pick up on the change to the ACS variable. For example draw a string on the normal statusbar and start with the full screen. Set the variable to a dynamic string and then switch to the normal status bar. SBarInfo will try to fetch the string on the tick screenblocks changes, does your patch account for that?
I think the proper solution would be to add support for drawing character arrays with SBarInfo.