EpisodeBrowser - GUI interface to locally stored TV series

Age	Commit message (Collapse)	Author
2022-07-20	Simplify UpdateLayout.	John Ankarström

2022-07-20	Simplify require, prefer.	John Ankarström

2022-07-20	Fix ANSI compatibility.	John Ankarström
	It's not very useful, but it's a fun exercise.
2022-07-20	Fix bug in and rename throw_nil, warn_nil.	John Ankarström
	In warn_nil, the return value was undefined on exception -- I think. While informative, the names throw_nil and warn_nil don't work very well in conditionals: if (warn_nil<f>(...)) g(); sounds like g should be called if f returns nil and a warning is issued. But it is actually the other way around; g is called if f is successful. if (prefer<f>(...)) g(); sounds less like that.
2022-07-19	Check Windows API calls for errors more consistently.	John Ankarström
	Some of the checks are likely redundant, but the Windows API documentation rarely makes it clear WHICH errors may be returned (and under which circumstances) rather than simply WHETHER errors may be returned (under any circumstances, including those that do not apply in the given case).
2022-07-19	Use static member function instead of friend function.	John Ankarström

2022-07-19	Use Prolog string buffer marks.	John Ankarström
	Speaking of unclear documentation, it is not obvious whether it is necessary for programs calling into Prolog to manually mark and release strings. I suppose that it should be, if the same logic that applies to terms apply to strings. On the other hand, the stack in which the strings are stored belongs to Prolog, and there is nothing that would prevent Prolog from cleaning up the strings when called at a later time. I am not sure. But better safe than sorry, I guess. The Mark class acts like the Frame class. The constructor and destructor are equivalent to the PL_STRINGS_MARK and PL_STRINGS_RELEASE macros. Unlike for 34c3280, I did not notice any differences in memory usage after this change. Perhaps that is because it has no effect; perhaps it is because Prolog's stack is very big.
2022-07-18	Fix typo.	John Ankarström

2022-07-17	Make ListView hWnd public.	John Ankarström
	A getter offers encapsulation, but it is also less transparent in a sense. Thinking of ListView as a struct, it is natural to expose hWnd as a public member variable.
2022-07-17	Update type names and variable prefixes.	John Ankarström

2022-07-17	Add const to pointers, update spacing.	John Ankarström
	It seems that "right-spaced" pointers are more widely used among C++ programmers.
2022-07-17	Add const to places.	John Ankarström
	Note that I did NOT add const to non-pointer/non-reference arguments in function declarations (without a following definition), as they do not mean anything there.
2022-07-17	Use SWI-Prolog's wide-character functions.	John Ankarström
	This avoids the use of TsmFromSz.
2022-07-16	Formatting.	John Ankarström

2022-07-16	Rewrite TszFromSz as TsmFromSz.	John Ankarström
	I.e. using std::basic_string<TCHAR> instead of TCHAR *. This removes all unmanaged frees.
2022-07-16	Add m_hWndParent to ListView.	John Ankarström
	This avoids g_hWnd.
2022-07-15	Split defs.h into separate header files.	John Ankarström
	This is feasible now that the makedeps script exists to automatically manage build dependencies (see 6034fe2, d00f8b3).
2022-07-15	Reimplement Pl with variadic templates	John Ankarström
	This is a bit safer and about as complex.
2022-07-15	Change declaration style.	John Ankarström

2022-07-14	Minor formal changes.	John Ankarström
	Height(DLVSIKEY) was incorrect. The argument to Height is supposed to be a boolean value, in this case false. It happened to work because DLVSIKEY is 0 (because Key is the 0th column in the data list view).
2022-07-14	Initialize global variables in the right place.	John Ankarström
	The rules for what messages are sent at window creation -- and in which order -- are neither intuitive or clear. WM_CREATE can NOT be relied upon to initialize global state required by handlers of other messages (such as WM_GETMINMAXINFO, which seems to be sent before WM_CREATE). Thus, the better solution is to initialize everything using a CBT hook before the window procedure is even run. Because CBTProc creates (child): windows of its own, though, one must be careful to only run the initialization once, which is done by checking g_hWnd.
2022-07-11	Improve ElvSort.	John Ankarström

2022-07-11	Refactor, adjust minimum window size.	John Ankarström

2022-07-10	Convert to C++.	John Ankarström
	I already hit upon some object-oriented programming patterns in *listview.c, so I felt that it would be natural to use this as an opportunity to learn C++.