Panduan gaya kode

Gaya kode HIDL menyerupai kode C++ dalam framework Android, dengan indent 4 spasi dan nama file campuran huruf besar/kecil. Deklarasi, impor, dan docstring paket mirip dengan yang ada di Java, dengan sedikit modifikasi.

Contoh berikut untuk IFoo.hal dan types.hal mengilustrasikan gaya kode HIDL dan memberikan link cepat ke detail tentang setiap gaya (IFooClientCallback.hal, IBar.hal, dan IBaz.hal telah dihilangkan).

hardware/interfaces/foo/1.0/IFoo.hal
/*
 * (License Notice)
 */

package android.hardware.foo@1.0;

import android.hardware.bar@1.0::IBar;

import IBaz;
import IFooClientCallback;

/**
 * IFoo is an interface that…
 */
interface IFoo {

    /**
     * This is a multiline docstring.
     *
     * @return result 0 if successful, nonzero otherwise.
     */
     foo() generates (FooStatus result);

    /**
     * Restart controller by power cycle.
     *
     * @param bar callback interface that…
     * @return result 0 if successful, nonzero otherwise.
     */
    powerCycle(IBar bar) generates (FooStatus result);

    /** Single line docstring. */
    baz();


    /**
     * The bar function.
     *
     * @param clientCallback callback after function is called
     * @param baz related baz object
     * @param data input data blob
     */
    bar(IFooClientCallback clientCallback,
        IBaz baz,
        FooData data);

};
hardware/interfaces/foo/1.0/types.hal
/*
 * (License Notice)
 */

package android.hardware.foo@1.0;

/** Replied status. */
enum Status : int32_t {
    OK,
    /* invalid arguments */
    ERR_ARG,
    /* note, no transport related errors */
    ERR_UNKNOWN = -1,
};

struct ArgData {
    int32_t[20]  someArray;
    vec<uint8_t> data;
};

Konvensi penamaan

Nama fungsi, nama variabel, dan nama file harus deskriptif; hindari singkatan yang berlebihan. Perlakukan akronim sebagai kata (misalnya, gunakan INfc, bukan INFC).

Struktur direktori dan penamaan file

Struktur direktori akan muncul sebagai berikut:

  • ROOT-DIRECTORY
    • MODULE
      • SUBMODULE (opsional, dapat lebih dari satu tingkat)
        • VERSION
          • Android.mk
          • IINTERFACE_1.hal
          • IINTERFACE_2.hal
          • …
          • IINTERFACE_N.hal
          • types.hal (opsional)

Dalam hal ini:

  • ROOT-DIRECTORY adalah:
    • hardware/interfaces untuk paket HIDL inti.
    • vendor/VENDOR/interfaces untuk paket vendor, dengan VENDOR merujuk pada vendor SoC atau OEM/ODM.
  • MODULE harus berupa satu kata kecil yang menjelaskan subsistem (misalnya, nfc). Jika lebih dari satu kata diperlukan, gunakan SUBMODULE bertingkat. Dapat ada lebih dari satu tingkat penyematan.
  • VERSION harus berupa versi yang sama persis (major.minor) seperti yang dijelaskan dalam Versi.
  • IINTERFACE_X harus berupa nama antarmuka dengan UpperCamelCase/PascalCase (misalnya, INfc) seperti yang dijelaskan dalam Nama antarmuka.

Contoh:

  • hardware/interfaces
    • nfc
      • 1.0
        • Android.mk
        • INfc.hal
        • INfcClientCallback.hal
        • types.hal

Catatan: Semua file harus memiliki izin yang tidak dapat dijalankan (di Git).

Nama paket

Nama paket harus menggunakan format nama yang sepenuhnya memenuhi syarat (FQN) berikut (disebut sebagai PACKAGE-NAME):

PACKAGE.MODULE[.SUBMODULE[.SUBMODULE[…]]]@VERSION

Dalam hal ini:

  • PACKAGE adalah paket yang dipetakan ke ROOT-DIRECTORY. Secara khusus, PACKAGE adalah:
    • android.hardware untuk paket HIDL inti (pemetaan ke hardware/interfaces).
    • vendor.VENDOR.hardware untuk paket vendor, dengan VENDOR merujuk ke vendor SoC atau OEM/ODM (memetakan ke vendor/VENDOR/interfaces).
  • MODULE[.SUBMODULE[.SUBMODULE[…]]]@VERSION adalah nama folder yang sama persis dalam struktur yang dijelaskan dalam Struktur direktori.
  • Nama paket harus berupa huruf kecil. Jika panjangnya lebih dari satu kata, kata tersebut harus digunakan sebagai submodul atau ditulis dalam snake_case.
  • Tidak boleh ada spasi.

FQN selalu digunakan dalam deklarasi paket.

Versi

Versi harus memiliki format berikut: 

MAJOR.MINOR

Versi MAJOR dan MINOR harus berupa bilangan bulat tunggal. HIDL menggunakan aturan versi semantik.

Import

Impor memiliki salah satu dari tiga format berikut:

  • Impor seluruh paket: import PACKAGE-NAME;
  • Impor sebagian: import PACKAGE-NAME::UDT; (atau, jika jenis yang diimpor berada dalam paket yang sama,import UDT;
  • Impor khusus jenis: import PACKAGE-NAME::types;

PACKAGE-NAME mengikuti format dalam Nama paket. types.hal paket saat ini (jika ada) otomatis diimpor (jangan impor secara eksplisit).

Nama yang sepenuhnya memenuhi syarat (FQN)

Gunakan nama yang sepenuhnya memenuhi syarat untuk impor jenis yang ditentukan pengguna hanya jika diperlukan. Hapus PACKAGE-NAME jika jenis impor berada dalam paket yang sama. FQN tidak boleh berisi spasi. Contoh nama yang sepenuhnya memenuhi syarat:

android.hardware.nfc@1.0::INfcClientCallback

Di file lain di bagian android.hardware.nfc@1.0, lihat antarmuka di atas sebagai INfcClientCallback. Jika tidak, hanya gunakan nama yang sepenuhnya memenuhi syarat.

Mengelompokkan dan mengurutkan impor

Gunakan baris kosong setelah deklarasi paket (sebelum impor). Setiap impor harus menempati satu baris dan tidak boleh diindentasi. Mengelompokkan impor dalam urutan berikut:

  1. Paket android.hardware lainnya (gunakan nama yang sepenuhnya memenuhi syarat).
  2. Paket vendor.VENDOR lainnya (gunakan nama yang sepenuhnya memenuhi syarat).
    • Setiap vendor harus berupa grup.
    • Urutkan vendor menurut abjad.
  3. Impor dari antarmuka lain dalam paket yang sama (gunakan nama sederhana).

Gunakan baris kosong di antara grup. Di dalam setiap grup, urutkan impor secara alfabetis. Contoh:

import android.hardware.nfc@1.0::INfc;
import android.hardware.nfc@1.0::INfcClientCallback;

/* Importing the whole module. */
import vendor.barvendor.bar@3.1;

import vendor.foovendor.foo@2.2::IFooBar;
import vendor.foovendor.foo@2.2::IFooFoo;

import IBar;
import IFoo;

Nama antarmuka

Nama antarmuka harus diawali dengan I, diikuti dengan nama UpperCamelCase/PascalCase. Antarmuka dengan nama IFoo harus ditentukan dalam file IFoo.hal. File ini hanya dapat berisi definisi untuk antarmuka IFoo (antarmuka INAME harus berada di INAME.hal).

Fungsi

Untuk nama fungsi, argumen, dan nama variabel yang ditampilkan, gunakan lowerCamelCase. Contoh:

open(INfcClientCallback clientCallback) generates (int32_t retVal);
oneway pingAlive(IFooCallback cb);

Nama kolom struct dan union

Untuk nama kolom struct atau union, gunakan lowerCamelCase. Contoh:

struct FooReply {
    vec<uint8_t> replyData;
}

Nama jenis

Nama jenis mengacu pada definisi struct atau union, definisi jenis enum, dan typedef. Untuk nama ini, gunakan UpperCamelCase/PascalCase. Contoh:

enum NfcStatus : int32_t {
    /*...*/
};
struct NfcData {
    /*...*/
};

Nilai enum

Nilai enum harus berupa UPPER_CASE_WITH_UNDERSCORES. Saat meneruskan nilai enum sebagai argumen fungsi dan menampilkannya sebagai nilai yang ditampilkan fungsi, gunakan jenis enum yang sebenarnya (bukan jenis bilangan bulat yang mendasarinya). Contoh:

enum NfcStatus : int32_t {
    HAL_NFC_STATUS_OK               = 0,
    HAL_NFC_STATUS_FAILED           = 1,
    HAL_NFC_STATUS_ERR_TRANSPORT    = 2,
    HAL_NFC_STATUS_ERR_CMD_TIMEOUT  = 3,
    HAL_NFC_STATUS_REFUSED          = 4
};

Catatan: Jenis dasar dari jenis enum dideklarasikan secara eksplisit setelah titik dua. Karena tidak bergantung pada compiler, penggunaan jenis enum yang sebenarnya akan lebih jelas.

Untuk nama yang sepenuhnya memenuhi syarat untuk nilai enum, titik dua digunakan antara nama jenis enum dan nama nilai enum:

PACKAGE-NAME::UDT[.UDT[.UDT[…]]:ENUM_VALUE_NAME

Tidak boleh ada spasi di dalam nama yang sepenuhnya memenuhi syarat. Gunakan nama yang sepenuhnya memenuhi syarat hanya jika diperlukan dan hapus bagian yang tidak diperlukan. Contoh:

android.hardware.foo@1.0::IFoo.IFooInternal.FooEnum:ENUM_OK

Komentar

Untuk komentar satu baris, //, /* */, dan /** */ tidak masalah.

// This is a single line comment
/* This is also single line comment */
/** This is documentation comment */
  • Gunakan /* */ untuk komentar. Meskipun HIDL mendukung // untuk komentar, komentar tidak dianjurkan karena tidak muncul dalam output yang dihasilkan.
  • Gunakan /** */ untuk dokumentasi yang dihasilkan. Ini hanya dapat diterapkan ke deklarasi nilai enum, metode, kolom, dan jenis. Contoh: 
    /** Replied status */
enum TeleportStatus {
    /** Object entirely teleported. */
    OK              = 0,
    /** Methods return this if teleportation is not completed. */
    ERROR_TELEPORT  = 1,
    /**
     * Teleportation could not be completed due to an object
     * obstructing the path.
     */
    ERROR_OBJECT    = 2,
    ...
}
  • Mulai komentar multibaris dengan /** di baris terpisah. Gunakan * di awal setiap baris. Akhiri komentar dengan */ di baris terpisah, dengan meratakan tanda bintang. Contoh: 
    /**
 * My multi-line
 * comment
 */
  • Pemberitahuan pemberian lisensi dan log perubahan harus memulai baris baru dengan /* (satu tanda bintang), menggunakan * di awal setiap baris, dan menempatkan */ di baris terakhir dengan sendirinya (tanda bintang harus sejajar). Contoh: 
    /*
 * Copyright (C) 2017 The Android Open Source Project
 * ...
 */

/*
 * Changelog:
 * ...
 */

Komentar file

Mulai setiap file dengan pemberitahuan pemberian lisensi yang sesuai. Untuk HAL inti, ini harus berupa lisensi Apache AOSP di development/docs/copyright-templates/c.txt. Jangan lupa untuk memperbarui tahun dan menggunakan komentar multibaris gaya /* */ seperti yang dijelaskan di atas.

Anda dapat menempatkan baris kosong setelah pemberitahuan lisensi, diikuti dengan informasi log perubahan/versi secara opsional. Gunakan komentar multibaris gaya /* */ seperti yang dijelaskan di atas, tempatkan baris kosong setelah log perubahan, lalu ikuti dengan deklarasi paket.

Komentar TODO

TODO harus menyertakan string TODO dalam huruf kapital semua, diikuti dengan titik koma. Contoh:

// TODO: remove this code before foo is checked in.

Komentar TODO hanya diizinkan selama pengembangan; komentar tersebut tidak boleh ada di antarmuka yang dipublikasikan.

Komentar antarmuka dan fungsi (docstring)

Gunakan /** */ untuk docstring multibaris dan satu baris. Jangan gunakan // untuk docstring.

Docstring untuk antarmuka harus menjelaskan mekanisme umum antarmuka, alasan desain, tujuan, dll. Docstring untuk fungsi harus spesifik untuk fungsi tersebut (dokumentasi tingkat paket berada dalam file README di direktori paket).

/**
 * IFooController is the controller for foos.
 */
interface IFooController {
    /**
     * Opens the controller.
     *
     * @return status HAL_FOO_OK if successful.
     */
    open() generates (FooStatus status);

    /** Close the controller. */
    close();
};

Anda harus menambahkan @param dan @return untuk setiap parameter/nilai yang ditampilkan:

  • @param harus ditambahkan untuk setiap parameter. Ini harus diikuti dengan nama parameter, lalu docstring.
  • @return harus ditambahkan untuk setiap nilai yang ditampilkan. Ini harus diikuti dengan nama nilai yang ditampilkan, lalu docstring.

Contoh:

/**
 * Explain what foo does.
 *
 * @param arg1 explain what arg1 is
 * @param arg2 explain what arg2 is
 * @return ret1 explain what ret1 is
 * @return ret2 explain what ret2 is
 */
foo(T arg1, T arg2) generates (S ret1, S ret2);

Aturan pemformatan

Aturan pemformatan umum meliputi:

  • Panjang baris. Setiap baris teks harus memiliki panjang maksimal 100 kolom.
  • Spasi kosong. Tidak ada spasi kosong di akhir baris; baris kosong tidak boleh berisi spasi kosong.
  • Ruang vs. tab. Hanya gunakan spasi.
  • Ukuran indentasi. Gunakan 4 spasi untuk blok dan 8 spasi untuk penggabungan baris
  • Penyangga. Kecuali untuk nilai anotasi, kurung kurawal buka berada di baris yang sama dengan kode sebelumnya, tetapi kurung kurawal tutup dan titik koma berikutnya menempati seluruh baris. Contoh: 
    interface INfc {
    close();
};

Deklarasi paket

Deklarasi paket harus berada di bagian atas file setelah pemberitahuan lisensi, harus menempati seluruh baris, dan tidak boleh diindentasi. Paket dideklarasikan menggunakan format berikut (untuk pemformatan nama, lihat Nama paket):

package PACKAGE-NAME;

Contoh:

package android.hardware.nfc@1.0;

Deklarasi fungsi

Nama fungsi, parameter, generates, dan nilai return harus berada di baris yang sama jika sesuai. Contoh:

interface IFoo {
    /** ... */
    easyMethod(int32_t data) generates (int32_t result);
};

Jika tidak muat di baris yang sama, coba tempatkan parameter dan nilai yang ditampilkan pada tingkat indentasi yang sama dan bedakan generate untuk membantu pembaca melihat parameter dan nilai yang ditampilkan dengan cepat. Contoh:

interface IFoo {
    suchALongMethodThatCannotFitInOneLine(int32_t theFirstVeryLongParameter,
                                          int32_t anotherVeryLongParameter);
    anEvenLongerMethodThatCannotFitInOneLine(int32_t theFirstLongParameter,
                                             int32_t anotherVeryLongParameter)
                                  generates (int32_t theFirstReturnValue,
                                             int32_t anotherReturnValue);
    superSuperSuperSuperSuperSuperSuperLongMethodThatYouWillHateToType(
            int32_t theFirstVeryLongParameter, // 8 spaces
            int32_t anotherVeryLongParameter
        ) generates (
            int32_t theFirstReturnValue,
            int32_t anotherReturnValue
        );
    /* method name is even shorter than 'generates' */
    foobar(AReallyReallyLongType aReallyReallyLongParameter,
           AReallyReallyLongType anotherReallyReallyLongParameter)
        generates (ASuperLongType aSuperLongReturnValue, // 4 spaces
                   ASuperLongType anotherSuperLongReturnValue);
}

Detail tambahan:

  • Tanda kurung buka selalu berada di baris yang sama dengan nama fungsi.
  • Tidak ada spasi antara nama fungsi dan tanda kurung buka.
  • Tidak ada spasi di antara tanda kurung dan parameter kecuali jika ada feed baris di antaranya.
  • Jika generates berada di baris yang sama dengan tanda kurung tutup sebelumnya, gunakan spasi sebelumnya. Jika generates berada di baris yang sama dengan tanda kurung buka berikutnya, ikuti dengan spasi.
  • Selaraskan semua parameter dan nilai yang ditampilkan (jika memungkinkan).
  • Indentasi default adalah 4 spasi.
  • Parameter yang digabungkan disejajarkan dengan parameter pertama di baris sebelumnya, jika tidak, parameter tersebut memiliki indentasi 8 spasi.

Anotasi

Gunakan format berikut untuk anotasi:

@annotate(keyword = value, keyword = {value, value, value})

Urutkan anotasi dalam urutan abjad, dan gunakan spasi di sekitar tanda sama dengan. Contoh:

@callflow(key = value)
@entry
@exit

Pastikan anotasi menempati seluruh baris. Contoh:

/* Good */
@entry
@exit

/* Bad */
@entry @exit

Jika anotasi tidak dapat muat dalam baris yang sama, indentasi dengan 8 spasi. Contoh:

@annotate(
        keyword = value,
        keyword = {
                value,
                value
        },
        keyword = value)

Jika seluruh array nilai tidak dapat muat dalam baris yang sama, masukkan baris baru setelah tanda kurung buka { dan setelah setiap koma di dalam array. Tempatkan tanda kurung tutup tepat setelah nilai terakhir. Jangan masukkan tanda kurung jika hanya ada satu nilai.

Jika seluruh array nilai dapat muat dalam baris yang sama, jangan gunakan spasi setelah tanda kurung buka dan sebelum tanda kurung tutup, serta gunakan satu spasi setelah setiap koma. Contoh:

/* Good */
@callflow(key = {"val", "val"})

/* Bad */
@callflow(key = { "val","val" })

Tidak boleh ada baris kosong antara anotasi dan deklarasi fungsi. Contoh:

/* Good */
@entry
foo();

/* Bad */
@entry

foo();

Deklarasi enum

Gunakan aturan berikut untuk deklarasi enum:

  • Jika deklarasi enum dibagikan dengan paket lain, tempatkan deklarasi di types.hal, bukan menyematkannya di dalam antarmuka.
  • Gunakan spasi sebelum dan setelah titik dua, serta spasi setelah jenis yang mendasarinya sebelum kurung kurawal terbuka.
  • Nilai enum terakhir mungkin tidak memiliki koma tambahan.

Deklarasi struct

Gunakan aturan berikut untuk deklarasi struct:

  • Jika deklarasi struct dibagikan dengan paket lain, tempatkan deklarasi di types.hal, bukan menyematkannya di dalam antarmuka.
  • Gunakan spasi setelah nama jenis struct sebelum kurung kurawal terbuka.
  • Sejajarkan nama kolom (opsional). Contoh: 
    struct MyStruct {
    vec<uint8_t>   data;
    int32_t        someInt;
}

Deklarasi array

Jangan masukkan spasi di antara hal berikut:

  • Jenis elemen dan tanda kurung siku terbuka.
  • Buka kurung siku dan ukuran array.
  • Ukuran array dan kurung siku tutup.
  • Kurung siku tutup dan kurung siku terbuka berikutnya, jika ada lebih dari satu dimensi.

Contoh:

/* Good */
int32_t[5] array;

/* Good */
int32_t[5][6] multiDimArray;

/* Bad */
int32_t [ 5 ] [ 6 ] array;

Vektor

Jangan masukkan spasi di antara hal berikut:

  • vec dan kurung sudut buka.
  • Kurung sudut buka dan jenis elemen (Pengecualian: jenis elemen juga merupakan vec).
  • Jenis elemen dan tanda kurung sudut tutup (Pengecualian: jenis elemen juga merupakan vec).

Contoh:

/* Good */
vec<int32_t> array;

/* Good */
vec<vec<int32_t>> array;

/* Good */
vec< vec<int32_t> > array;

/* Bad */
vec < int32_t > array;

/* Bad */
vec < vec < int32_t > > array;