Store the index of the member in MemberAccess

Fix mismatched new[]/delete

Rearrange some utility struct constructors

Guard against hitting the end of the array in get_gl_type

Comments and cosmetic cleanups

Do an early return from Mesh::draw if there's no data

Otherwise if would try to set zero-sized storage for the buffers, which
is not allowed.

Also guard against empty batches in resize_buffers just in case.

Use uniform block and texture bindings from GLSL

Add a target version option to the command-line GLSL compiler

Better way to request features for different GLSL versions

Assign bindings to all uniform blocks and sampler uniforms

Add legacy conversion for binding layout qualifiers

These are not yet used by Program because there's no good way to detect
which uniforms had bindings in case the implementation does support the
qualifiers.  Anything missing a binding gets assigned binding 0, which
is also a valid manually specified binding.  Upcoming commits will move
bindings entirely to GLSL side.

Store constant id limit as range, not max value

Better checking of where the binding qualifier is allowed

Disallow the descriptor set qualifier for now

It's only meaningful for Vulkan, which isn't implemented yet

Allocate locations to interface variables

Improve compatibility checks of location qualifiers in various contexts

Even more validation for uniform mismatches

Validate location overlap and type matching for GLSL interfaces

Also allocate a buffer when mapping it

Fix a misformatted branch in GLSL compiler debug output

Store program resource information in vectors instead of maps

Store name in attributes collected from SPIR-V module

Fix an error in calculating SPIR-V uniform offsets

Rename some things in Program

Clear all maps before linking the Program

Add default constructors for the info structs in Program

Make constant id range configurable and avoid duplicates

GlslangValidator apparently only allows IDs up to 2047.  I've yet to find
out if there's a limit in SPIR-V itself.

Split up ComponentSpecializer

It was weird having the same class do two things, and also those things
should be done at slightly different points of the compilation process.

Fix extension declarations in GLSL output

Split glsl/generate.cpp in two

Mark generated array types as internal

Fix layout processing for old GLSL versions

layout(location) is not supported in 1.10 and 1.20, even with the
ARB_explicit_attrib_location extension.

Add missing NodeRemover invocation to LegacyConverter

Refactor layout legacy conversion code to be more extensible

Use type information in converting legacy texture sampling functions

Fill in the required version if empty

Most of the version checks won't trigger for GLSL 1.20 and none will for
GLSL 1.10.

Combine the precision handling visitors and rewrite the logic

It had fallen befind the times and still used the textual type
information.  The logic of what to do is now also contained in the
visitor.

Rename compatibility.* to finalize.*

This seems more appropriate for some SPIR-V stuff I'm going to add.

Always remove location qualifier if not supported

Move LegacyConverter invocation to the finalize stage

Add support for loading SPIR-V shaders

Rename size to array_size in Program variable info structs

Add a compile mode flag to the command-line GLSL compiler

Use DataType instead of GLenum for Program variable types

Add data types for vectors, matrices, images and samplers

Update gl.xml to latest version

Add some precondition checks to Program

Adding stages from multiple modules presents some difficulties in matching
the interfaces.

Split Module into a base class and format-specific class

Refactor adding and compiling shader stages in Program

SPIR-V stages don't need to be compiled so it makes more sense to compile
when adding stages from GLSL.

Track allocation status of Buffer

If ABR_buffer_storage is not supported, calling storage() and then
sub_data() did not result in the buffer being allocated.

Make Text more flexible about which texture slot it uses

Different material types have different names for their textures so
using a fixed name here does not cut it anymore.

Use validate_name when copying ProgramData

It makes sense to use the same debug build logic here.

Avoid accessing a null object if there's only a sampler for some unit

Fix loading .tex2d files with ResourceManager

The texture's loader already informs the resource manager of the location
of the image, so it shouldn't be done here.  There's also no need to load
the image first anymore.

Move checking of vertex attribute indices to VertexSetup

This avoids issues when loading meshes with ResourceManager, since it's
not possible to read the value of GL_MAX_VERTEX_ATTRIBS from the loading
thread.

Refactor lexical conversion of vertex attributes

There was an error in the size of the base string for the GENERIC
attribute, causing it to never match.  Using a utility function makes
it less dependent on magic numbers.

Color is still handled specially because of the type suffix.

Fix TextureCube mipmap level allocation tracking

Export texture extension modes from Blender

Add CLAMP_TO_BORDER and border color to Sampler

Support defining uniform values for materials in Blender

Rename material map to material atlas in the exporter

Fix handling of techniques in object exporter

Materials with external rendering mode don't get technique resources
generated them, so it's better to use the technique name here.

Fix exporting of materials without any textures

If a generator expression produces no items, trying to access __next__()
produces a StopIteration exception.  It's better to create a list so we
can check if it's empty.

Fix and improve vertex group support

Some parts of the export code had gotten out of sync in 306be12.

Groups and weights now have dedicated attributes instead of using an
arbitrary generic attribute.  They're also separate attributes,
allowing up to four groups per vertex.

Add support for exporting vertex colors

Refactor the mesh exporter to use UVs directly from UV layers

Fix suspicious code in the mesh exporter

The pivot_vertices function was expecting a list of vertices, but only
one vertex was passed in.  In fact the entire function seemed rather
useless now that the mesh is triangulated at an early stage.

Update the desert pillars demo to work once again

There's some warnings which require a more comprehensive overhaul to
get rid of.

Add a collection parameter to RenderPass::set_material

If the pass did not already have a shader, it will try to create one from
the material and fail if it has no collection.

Rearrange vertex attributes

Since fixed function is no longer supported, I don't have to care what
attribute number Nvidia gives the fixed function attributes.  The old
layout had weird holes and made custom attributes hard to add; now the
free attributes are neatly at the end.

The duality between standard and generic attributes has been removed.
User-defined attributes now have their own range.  No deprecated
versions have been kept since they would be difficult to support
properly.

Rename VertexComponent to VertexAttribute

Also rename the related functions.  Get_component_type and
get_component_size in particular were problematic since pixelformat.h
uses the same names, and the ones in vertexformat.h don't take an enum
type.

Refactor vertex builders

All attributes are now stored in a single vector.

Rearrange various uniforms to be declared in the file which uses them

Support exporting unlit materials

These use the emission surface type in Blender.  While that may seem
strange at first, it's actually a good match to how unlit materials
behave in a realtime scene.

Refactor material property handling in the Blender exporter

Adjust uniform organization for UnlitMaterial

Its only property are now stored in a struct to match other material
types.  This allows using the name "color" without conflicting with the
vertex attribute.  The texture uniform was also renamed in order to
avoid clashing with the texture() function.

Decouple DataType from the OpenGL constants

This allows defining the values so that manipulation of related types is
easier.  And I can't keep relying on the GL values anyway, with the goal
of adding a Vulkan backend.

Fix function inlining regressions

Fix test case errors revealed by the previous commit

And one case of adjusted error message too.

Add a bunch of validation for declarations in GLSL

This is why I renamed the old DeclarationValidator a few commits ago,
because that name is a better fit here.

Some touch-up of the debug dump

Since nodes are now easily distinguished, there's no need for special
labeling branches in many cases.

Record correct source information in interface layout statements

Show node source locations in AST dump

Rename DeclarationValidator to IdentifierValidator

This name more accurately describes what it does.

Support layout qualifiers on GLSL interface blocks

Rename InterfaceBlock::name to block_name for clarity

Add push_back_nocopy to NodeContainer for clarity

It's not immediately obvious that pushing null and then assigning avoids
cloning the node.

Add idempotence test cases for GLSL compiler

Already compiled GLSL code should not change when passed through the
compiler again.

Refactor the GLSL compiler test cases with a helper class

I'm not super pleased with the virtual fail function in the helper, but
no better ideas came to mind so it can be like that for now.

Always show shader compiler diagnostics in debug builds

Forbid certain operations in SL::Compiler if compilation isn't done

In particular, Formatter can't deal with unresolved declarations.

Adjust naming of generated variables

Using the inlined function name as a prefix easily produces inconsistent
results, so better to just use numbers after the name.

Test case tweaks

Support inlining GLSL functions with parameters

Refactor variable renaming in InlineContentInjector once again

It should now be even better at recognizing conflicts, and avoid renaming
when not necessary.

Use a temporary staging block in InlineContentInjector

This allows the return variable name generation to avoid other names from
the source function, such as a return variable from previous inlining.

Remove an obsolete variable

This hasn't been used since 0388083.

Don't modify the target block's variable map from InlineContentInjector

This felt hacky, and also seemed wrong since the variable map contains
the original names of the variables, not new ones.  Instead avoid
inlining more than one function per pass into any given function.

Check that the types of ternary branches exist before accessing them

This fixes a bug introduced in 3a1fe83 and triggered by 5e4204e.

Eliminate constant ternary expressions