All sources were moved to external/qemu/distrib/android-emugl
to make it easier to modify both the emulator and host libraries
at the same time. See:

    https://android-review.googlesource.com/#/c/118203/

As such, the sources in this directory are now ignored and can be
safely removed.

Change-Id: I5d143c971f995e2599e5e1d6174030d6803e8080

This patch fixes the parsing of type declarations and parameter
declarations to:

- Properly recognize complicated types like 'const int* const*'
  which previously required omitting spaces (e.g. 'const int*const*')
  to be recognized by the parser.

- Normalize the type strings (e.g. 'const char **items' -> 'const char** items')

- Add a unit test program (emugen_unittests) to check emugen's internal
  functions. For now this only applies to the new functions introduced
  in the new header Parser.h

+ Update emugen test suite accordingly.

Change-Id: Ib1b6bcbae97e1ee7d8b272843dfb5926d2d98fd2

This adds a new flag that can be applied to entry points for 'emugen',
named 'flushOnEncode'. When used, the generated encoder will call
stream->flush() just after adding bytes to the stream.

This is needed to match the manual change that was performed in
the renderControl encoder in the following patch:

    https://android-review.googlesource.com/#/c/95864/

Change-Id: I8cc8fdb0d38ef27e8ba646c83d717166400babfd

This patch ensures that the auto-generated client_context.h
headers do not generate compiler warnings in the setError()
function.

+ Remove tiny warning for NativeLinuxSubWindow.cpp

Change-Id: Ibab92ab3332fd284589435732b52c011ae21c15f

This patch ensures the generated sources are cleaned up a little
bit, more specifically:

  - Add proper end-ifdef-guard comments.
  - Use anonymous C++ namespaces to avoid name conflicts in the encoder.
  - Remove extra spaces / empty lines / indent.
  - Use 'const' when defining constant tables.

Change-Id: Ib11fd06adb9075d472d1dd2fd6defb0760aaa2c2

This adds a small test suite to check the output of the 'emugen'
program. This serves two purposes:

  1) To more easily check that modifications to 'emugen' do not
     break stuff liberally.

  2) To better document how the changes in the generator actually
     modify the output.

To run it, simply call the 'run-tests.sh' script with 'emugen' in
your path, or use --emugen=<program> otherwise. See --help for more
details.

NOTE: The test suite currently doesn't check that the generated
      sources compile properly, or that they even work as expected.

See the following external/qemu patch to call run-tests.sh during
each android-rebuild.sh run:

    https://android-review.googlesource.com/112541

Change-Id: I9abc3f9ae63b0bb753f0f8e07c1b3f0b11a3252f

Modify the generated decoder to extract paramters from the
stream into local variables, the rules are:

  - non-pointer parameters are extracted into local variables
    named "var_<name>", where <name> is the parameter name as
    it appears in the protocol specification for the entry.

  - for input pointers, use "inptr_<name>", as well as
    "size_<name>" for the corresponding size in bytes.

  - for output pointers, use "outptr_<name>" and "size_<name>"

This makes the generated code easier to understand, for example
the following:

	case OP_glBindAttribLocation: {
		size_t tmpPtr2Size = (size_t)*(uint32_t *)(ptr + 8 + 4 + 4);
		InputBuffer tmpPtr2(ptr + 8 + 4 + 4 + 4, tmpPtr2Size);
		DEBUG("gl2(%p): glBindAttribLocation(%u %u %p(%u) )\n", stream,Unpack<GLuint,uint32_t>(ptr + 8), Unpack<GLuint,uint32_t>(ptr + 8 + 4), (const GLchar*)(tmpPtr2.get()), (uint32_t)tmpPtr2Size);
		this->glBindAttribLocation(Unpack<GLuint,uint32_t>(ptr + 8), Unpack<GLuint,uint32_t>(ptr + 8 + 4), (const GLchar*)(tmpPtr2.get()));
		SET_LASTCALL("glBindAttribLocation");
		break;
	}

becomes:

	case OP_glBindAttribLocation: {
		GLuint var_program = Unpack<GLuint,uint32_t>(ptr + 8);
		GLuint var_index = Unpack<GLuint,uint32_t>(ptr + 8 + 4);
		uint32_t size_name = Unpack<uint32_t,uint32_t>(ptr + 8 + 4 + 4);
		InputBuffer inptr_name(ptr + 8 + 4 + 4 + 4, size_name);
		DEBUG("gl2(%p): glBindAttribLocation(%u %u %p(%u) )\n", stream,var_program, var_index, (const GLchar*)(inptr_name.get()), size_name);
		this->glBindAttribLocation(var_program, var_index, (const GLchar*)(inptr_name.get()));
		SET_LASTCALL("glBindAttribLocation");
		break;
	}

Change-Id: Ifa8c73eec85b818d6d8b6371587da9fdfa57de8e

Minor patch that ensures that the pointer/position increment all
happen at one place, instead of being replicated in each case
statement. In other words, the generated code used to look like:

   switch (opcode) {
   case OP_someOperation1: {
       ...
       ptr += *(uint32_t)(ptr + 4);
       pos += *(uint32_t)(ptr + 4);
       break;
   }
   case OP_someOperation2: {
       ...
       ptr += *(uint32_t)(ptr + 4);
       pos += *(uint32_t)(ptr + 4);
       break;
   }
   ...
   default: {
       unknownOpcode = true;
   }

Now it looks like:

   switch (opcode) {
   case OP_someOperation1: {
       ...
       break;
   }
   case OP_someOperation2: {
       ...
       break;
   }
   ...
   default: {
       unknownOpcode = true;
   }

   if (!unknownOpcode) {
      ptr += packetLen;
      pos += packetLen;
   }

Which is cleaner. Also change the type of |opcode| and |packetLen|
to uint32_t and size_t respectively. + Minor formatting changes to
indentation.

Change-Id: If0002fe18a24b9ce6691e3e3cd3e102d1e00d4c9

This patch introduces two helper macros in the generated
decoder to make its source code slightly more readable
(DEBUG and SET_LASCALL).

Change-Id: I6659fe69a5533e5194d8db5f1882abaf1a5daee5

This patch simplifies the generated encoders and decoders by
getting rid of the accessor functions (e.g. set_glDrawElementsData)
given that all fields are public and can be written to directly.

Change-Id: I15f4caac95e4d5f1e24a1a5838622600c6bc3207