Forum: Bookworm

Initial changes to support MODE7 graphics natively

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:17

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6376

typo: terminal

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:17

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6377

I'm not seeing the string Mode7TerminalQ used in the C++ source files, so we should remove this (if unused).

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6378

Remove this line

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6379

I'm uneasy about all this copy-pasta. Now we've got 2 places that someone (me) has to update anytime we need to update these defaults. Please move to a common source file or .ini file or something.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6381

Similar to chat_sec.js, this is almost identical to exec/chat_sec.js, so let's merge the differences and get rid of this almost-identical copy/variant.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6380

This file looks almost identical to chat_sec.js, so let's merge the differences and get rid of this file.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6383

Delete extraneous white-space change.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6382

On second thought, this should be a module option. :-)

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6385

Update comments

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6384

Delete extraneous white-space change.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6386

I'm thinking that yesnobar.js and noyesbar.js should use the same set of options from modopts.ini - so just pass "yesnobar" here.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:18

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6387

So this would be changed `options.yesno_question`

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:19

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6388

This function doesn't appear to do anything - so not needed?

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:19

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6389

What are we using from mode7defs.h in this file?

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:19

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6390

Is there a reason to over-ride the default_term_height here? Is a 24 row mode7 terminal just never possible?

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:19

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6391

What are we using from mode7defs.h here?

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:19

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6392

is .m7 the best file suffix/extension to use for BBC mode7 encoded files? I don't know, just curious/asking.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:19

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6393

I noticed this conversion isn't happening in the Windows version of this function, but it's not happening for PETSCII or UTF-8 either, so we'll have to deal with all 3 of those charset conversion together someday.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:19

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6394

Update comment.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:20

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6395

Typo

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:20

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6396

Remove cruft. :-)

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:50

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6398

This file appears to be identical to system.msg just with the Ctrl-A codes stripped. If we're stripping Ctrl-A codes from display files already, why do we need this file (the result is the same)?

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:50

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6397

Why's there a date in this file?

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:45:54

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6399

Extraneous '/' in this string.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:46:01

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6400

Maybe "BBC" instead? It's a little weird the charset and the terminal type are the same string value.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:46:03

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6401

Something wonky here with tabs vs spaces.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:46:05

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6402

Something weird going on here with tabs vs. spaces.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:46:06

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6403

tabs vs spaces

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:46:10

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6404

tabs vs spaces

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:46:13

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6405

Nice! It could be merged as is but it'd be nice if it was cleaned-up a bit first.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:46:55

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6406

Now that I think about it... where is the change to ignore/strip Ctrl-A codes? I'm not seeing that.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 20:55:16

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6407

I see now, none of the values from mode7defs.h appear to be used, so I'm guessing this branch isn't as far along as I assumed. Hopefully the comments so far are at least helpful in some regard.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Nigel Reed@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 21:01:27

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6408

Fixed at my end.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Nigel Reed@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 21:02:51

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6409

This will be added per my comment. I've not finished the autoterm stuff where it goes through and asks what sort of terminal you're using. I wanted to put these in so you're aware of them when adding to text.dat.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Nigel Reed@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 21:03:57

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6410

Removed. I can remove any others I come cross in the future since we're not longer using CVS.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Rob Swindell@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 21:06:29

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6411

Yup, but if we can unify chat_sec.js, that becomes N/A anyway.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

From Nigel Reed@VERT to GitLab note in main/sbbs on Tuesday, February 11, 2025 21:09:52

https://gitlab.synchro.net/main/sbbs/-/merge_requests/499#note_6412

OK. I will have to think about how I want to do that. I'll put it on my todo list.

---
� Synchronet � Vertrauen � Home of Synchronet � [vert/cvs/bbs].synchro.net

Who's Online
Recent Visitors
- Guest
  Thursday, May 08, 2025 17:18:18
  from Berlin via Telnet
- Guest
  Wednesday, May 07, 2025 23:37:51
  from A via Telnet
- G
  Tuesday, May 06, 2025 23:23:25
  from Za via SSH
- Guest
  Thursday, May 01, 2025 18:20:59
  from D via Telnet

System Info

Sysop:	marcvandam
Location:	Utrecht
Users:	11
Nodes:	4 (0 / 4)
Uptime:	191:19:20
Calls:	84
Calls today:	84
Files:	72
U/L today:	2 files (2,547K bytes)
D/L today:	376 files (271M bytes)
Messages:	11,285
Posted today:	7

Initial changes to support MODE7 graphics natively

Who's Online

Recent Visitors

System Info