GitLab

This allows C:tboxlibint to be removed from our command line include path.
Requires tboxlib!2

Version 0.45. Not tagged

Detail:
  Correction to argument order when getting the sprite size.
Admin:
  Spotted during work on the Git source control client.

Version 0.45. Tagged as 'Gadgets-0_45'

Fix checking of the DesktopColours flag to the scrolllist and textarea gadget SetColour methods - didn't match docs.
Fix for expression precedence of + and ? operators by sprinkling in some brackets.
Ref https://www.riscosopen.org/forum/forums/4/topics/9503#posts-72595

Also, optimise the forced redraw if the text area colour overall didn't change (reduces flicker).
Docs corrected to list RISC OS palette words in the right order.

Version 0.44. Tagged as 'Gadgets-0_44'

Admin:
  Converge on "ObjModName has active tasks.". This allows only 1 section describing the errors in the User Guide.

Version 0.43. Tagged as 'Gadgets-0_43'

Rather than leak 40 bytes by failing to free(), just use an automatic variable and let the compiler worry about function exits.
From a tip off in https://www.riscosopen.org/forum/forums/4/topics/3990

Version 0.42. Tagged as 'Gadgets-0_42'

When there are no items in the scroll list, the check of select/adjust clicks were using event_item=0 and looking up in a NULL sdata list.
Move the (existing) early exit a bit higher up, since there are no entries in the list the _select_item() calls aren't needed anyway.
Fixes issue reported by John Rickman https://www.riscosopen.org/forum/forums/4/topics/4011

Version 0.41. Tagged as 'Gadgets-0_41'

Both the TextArea gadget and ScrollList were outputting an unhelpful white rectangle and mysterious black rectangle when asked to plot the gadget prototype (eg. in the gadget palette in ResEd). The black rectangle turned out to represent a scroll bar, though it was the wrong width.
Changed to plot a 2D scrollbar (ala RISC OS 2) then overlay a virtual icon with the name of the gadget type.
main.c:
Relocate helper functions into glib.
Return 'Bad SWI' for unknown subreasons.
glib.c/glib3.c:
Remove various unused functions.
Added new function to plot a grey scrollbar from nested rectangles.
TextArea.c:
Add a vertical, or horixontal, or both, scrollbar plus the label "TextArea".
ScrollList.c:
Add a vertical scrollbar plus the label "ScrollList".

Requires Common-0_25 for the colour defines. Tested in !ResEd in various eigenfactor modes.

Version 0.40. Tagged as 'Gadgets-0_40'

When _set_colour() is called, remember to redraw the gadget if the colours changed.
When _set_font() is called, remember to redraw the gadget and try to preserve the scroll bar positions.
Save a couple of SWIs if we know the scroll bars must jump in 2 other places.
Remove ScrollLisS, its (never called) function did nothing but return.
Removed duplicate redraw_box, same as redraw_all_box and marked as const.

Reported by Fred Graute, reference
  https://www.riscosopen.org/forum/forums/4/topics/3628
the memory leak aspects of the report have been transferred to ticket #414 for later consideration.
Tested with !ExifEdit and some special versions from Fred with the redraw workarounds removed.

Version 0.39. Tagged as 'Gadgets-0_39'

Don't bother calling TinyStubs SWI since in ROM it does nothing, and the RAM versions haven't used it since ~1997.
Use module header constants generated from CMHG rather than hand made one.

Version 0.38. Tagged as 'Gadgets-0_38'

Supporting code borrowed from 'Iconbar' toolbox module.
Tested in a ROM build and RAM build, triggering one error message and checking the dynamic area in task manager.

Version 0.37. Tagged as 'Gadgets-0_37'

For some reason this toolbox module was unique in linking against the public toolboxlib rather than the internal one, this meant having to carefully tippy toe around not mixing header files (as they contain similar but differently named definitions).
A massive search and replace job.
Tested briefly using ResEd/ResTest and a textarea and scrolllist in a ROM build.
Requires Common-0_21.

Version 0.36. Tagged as 'Gadgets-0_36'

When 'select only one' or 'multiple selections forbidden' is in use any selection changes first deselect all.
However, in the second case, the module overrode the caller's selection reason so adjust clicking on an item would not deselect it - instead reselecting it!
Logic ammended.
From http://www.riscosopen.org/forum/forums/4/topics/1057

Version 0.35. Tagged as 'Gadgets-0_35'

The last change meant it was no longer being generated, so ammended the code to not need it in line with all the other toolbox modules.

Version 0.34. Tagged as 'Gadgets-0_34'

Removed local duplication of string32 and rmensure, since the common library has these in.
Rename TextGadget.c to main.c to fall into line with the "o.mainROM" rule.
Compiles, but not tested.

Version 0.33. Tagged as 'Gadgets-0_33'

Detail:
  c/ScrollList, c/TextArea - ScrollList_SetFont and TextArea_SetFont now accept font handles instead of just font names. The handles are handled in the same way as the Window module, i.e. the handle will be managed and freed by the TextGadgets modules. Fixes bug #212.
  c/ScrollList, c/Scrollbar, c/TextArea - Fix handling of Toolbox_ObjectDeleted events. Original code would corrupt the object lists when deleting items, and reallocate much more memory than was actually needed. Fixes bug #214
  c/MemMan - Fix grow_handlers to not leave handle_list as NULL if the realloc fails. Fixes bug #216.
  c/MemMan, c/TextArea, c/TextMan, h/MemMan, h/TextMan - Changed how the 'Handle' type is handled. Instead of MemMan giving out direct pointers to Handle instances, it now gives out identifiers (HandleId). These identifiers are then converted to Handle pointers via get_handle whenever the contents need to be accessed. This allows the handle_list to be safely realloc'd without breaking all the existing Handle pointers. Fixes bug #217
  c/MemMan - Reduced HDNLE_GRW_BY from 1000 to 128 to save a bit of memory
Admin:
  Module compiles and runs, but actual functionality is untested.
  Fixes bugs 212, 214, 216, 217.


Version 0.32. Tagged as 'Gadgets-0_32'

Previously, a number of errors were contructed as automatic variables then returned to the caller - potentially accessing stale stack area. All error blocks are now held in ROM and cast from there.
Error numbers are now not &00000000 or &20202020, using the allocated base - requires 'Common-0_15'.
Trimmed some unused debugging code out, saves about 3k.
Renamed C++ clashing variables to stop the compiler moaning.
Implemented feature request to accept the empty string "" as no font handle in _SetFont calls.
At some point, this module needs internationalising.

Version 0.31. Tagged as 'Gadgets-0_31'

Version 0.30. Tagged as 'Gadgets-0_30'

Detail:
Admin:   builds


Version 0.29. Tagged as 'Gadgets-0_29'

Detail:
  Was still doing link -bin in rom_link phase. Changed to use link -rmf
  for consistency with all other C components, and with the BBE auto-
  generated makefile. Also has the benefit of saving approx 1K of ROM space.
Admin:
  Not expected to cause any problems.

Version 0.28. Tagged as 'Gadgets-0_28'

 create res file with a single TextArea in it
 change flags to be auto create and auto display
 drop into restest
 delete the 8 character "TextArea" default text
 press enter
 press backspace
Also,there was a _swix(&107) left in from some point in the past.

Version 0.27. Tagged as 'Gadgets-0_27'

NEWDELETE switch like RISCOSLib is now).
Key 8 deletes left
Key 127 deletes right
The Copy key does nothing (home and end aren't implemented it seems).
Fix from Kevin for it claiming a Dynamic area of maximum size - greedy
guts now limited to 16M.

Version 0.26. Tagged as 'Gadgets-0_26'

Detail:
  debug.s was still in the build and included a header which had been removed
  in a previous version.  The code in debug.s in not used anywhere, so I've
  just removed it (and the corresponding debug.h, which was empty anyway).
Admin:
  Compiled.

Version 0.25. Tagged as 'Gadgets-0_25'

  Improved some of the rather odd Piers-assembler.
Detail:
  Some rather odd assembler has been tidied up.
  Removed some obsolete header files that are no longer required.
  Removed "objs" - nothing uses it - looks like an accidental
    checkin.
Admin:
  Tested on Risc PC with Phoebe & 32 machine builds.

Version 0.24. Tagged as 'Gadgets-0_24'

	Altered to work with HdrSrc 0.46

Detail:
=======
 	HdrSrc 0.46 now includes the macros defined in the hdr.Cvars local
file in it's Hdr:Macros file. Hence have altered this component to use the
global header rather than the local copy. Also altered it so its hdr.SWINames
file no longer attempts to define the AddSWI macro, which is also defined in
hdr:Macros.

Admin:
======
	tedted that it builds.

Version 0.23. Tagged as 'Gadgets-0_23'

Detail:
faded flag added to ScrollbarP.h, Scrollbar.c uses this to calculate extent.
scrollbar_fade function called as a PRIVATE_HANDLER from TextGadget.c
Admin:
Built and tested on Pete Goodliffe's Risc PC. Problem with CVS misusage fixed
(a couple of modifications were accidentally removed).

Version 0.22. Tagged as 'Gadgets-0_22'

* Fixed the textarea_set/get_cursor_position methods to use the method interface as specified in docs.textarea.  I wasn't returning values in R4 as it showed in the document, I am now.  The fetching of the cursor position (in both the get and set calls) doesn't appear to work (always returns 0) with what appears to be the correct code to achieve this.  The fixing of this is left as an exercise for the bored and very brave person...textarea internals are "interesting".

 * Added some description of the methods into docs.textarea


Version 0.21. Tagged as 'Gadgets-0_21'

The scrollbar gadget can now be 'faded' (it doesn't actually fade - it just stops responding and maximises the sausage).

   TextArea_SetCursorPosition and TextArea_GetCursorPosition.

 * Removed a lot of unneeded comments that wrapped around changes I made
   a while ago.

 * !MkRam now enables throwback.


Version 0.20. Tagged as 'Gadgets-0_20'

Changed scrollbar gadget to initialise title-bar validation string despite not having a title bar, to get around a crash in the wimp

Version 0.19. Tagged as 'Gadgets-0_19'

Version 0.18. Tagged as 'Gadgets-0_18'

An unsigned int was involved in a signed comparison (no compiler warning?) which led to empty lists contracting to zero height. Fixed.

Version 0.17. Tagged as 'Gadgets-0_17'

Added scrolllist_set_item_text. Tweak to extent setting to account for
window border. _add_item fix ('>' should've read '>=' in index range
check). Can now tell _add_item to make the new item visible (flag
ScrollList_AddItem_MakeVisible), else it will try to maintain the y
scroll position. _delete_items continues to jump to the top as before,
but you can ask it to try and maintain the y scroll position too
(flag ScrollList_DeleteItems_DoNotJumpToTop).

Version 0.16. Tagged as 'Gadgets-0_16'

Title background colour of gadgets was either undefined or set to 0; should
be 2. Corrected. Would be apparent with 2D tools or any transparent tool
sprite set.

SELECT clicks on deselected entries in a ScrollList now deselect any other
selected items before selecting the new one.

Embedded window of TextArea and ScrollList now put in the right place. At
create time, the sizes of the current tool sprite set is also checked; this
used to be hard coded. If the tool sizes and/or Desktop scaling changes at
run-time this won't be noticed yet, as no mechanism to pass Wimp messages
to *external* gadgets exists in the Toolbox currently. Still, at least it
gets it right the next time a new instance of the gadget is created.

Flag bit ScrollList_SelectionChangingMethod_SendEvent to any of
scrolllist_select_item, scrolllist_deselect_item or scrolllist_delete_items
prompts them to send ScrollList_Selection to the client on completion. This
can help simplify client UI code for UIs which depend on what is selected
(e.g. a "Select all" button greys out if all items are selected by a
keyboard shortcut - this can now all work on just the one event handler).

Flag bit ScrollList_SelectionChangingMethod_OnAll to either of
scrolllist_select_item or scrolllist_deselect_item tells the methods to
act like a "Select all" or "Clear selection" call. This is a common client
operation and it's a lot easier for the ScrollList gadget to do it than
have the client iterate over all items.

Supports new method ScrollList_CountItems (scrolllist_count_items veneer
present in tboxlibs 0.04 or later). Returns number of entries in gadget.
Avoids clumsy situation of both the gadget and the clienty keeping track
of how many entries are present.

Note that docs.ScrollList mis-documented some flags settings. Now fixed.

Version 0.15. Not tagged

Version 0.14. Tagged as 'Gadgets-0_14'

Version 0.13. Not tagged

Re-arranged header files to be suitable for sharing with Toolboxlib and
for export to RiscOS/Export/Lib/tboxlibs/h.  The ordering and style of
these files now matches other toolboxlib header files.

Private TextArea macro definitions moved from TextArea.h to TextAreaP.h

Most function declarations are now auto-generated by MethodGen so they
come with documentation for free, except for TextArea_GetSelection*
which need to be done manually.  This new set of declarations is
identical to the hand-written ones which existed previously.

Changed TextArea windows to be created with the NoBounds window flag set.

Version 0.11. Tagged as 'Gadgets-0_11'