summaryrefslogtreecommitdiffstats
path: root/HACKING
Commit message (Collapse)AuthorAgeFilesLines
* HACKING: List areas where we may rely on impdef C behaviourPeter Maydell2012-12-081-0/+20
| | | | | | | | | Add a section to HACKING saying which version of the C spec we use and describing the bits of implementation defined C compiler behaviour which C code in QEMU is allowed to rely on. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* Rename target_phys_addr_t to hwaddrAvi Kivity2012-10-231-1/+1
| | | | | | | | | | | | | | | target_phys_addr_t is unwieldly, violates the C standard (_t suffixes are reserved) and its purpose doesn't match the name (most target_phys_addr_t addresses are not target specific). Replace it with a finger-friendly, standards conformant hwaddr. Outstanding patchsets can be fixed up with the command git rebase -i --exec 'find -name "*.[ch]" | xargs s/target_phys_addr_t/hwaddr/g' origin Signed-off-by: Avi Kivity <avi@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* doc: update HACKING wrt strncpy/pstrcpyJim Meyering2012-10-051-4/+5
| | | | | | | | Reword the section on strncpy: its NUL-filling is important in some cases. Mention that pstrcpy's signature is different. Signed-off-by: Jim Meyering <meyering@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* HACKING: clarify allocation/free recommendationsPeter Maydell2011-12-151-4/+6
| | | | | | | | | | Clarify the allocation/free recommendations; this is mostly just tidying up following the global-search-and-replace done with the conversion to the GLib g_malloc and friends. Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* Convert last qemu_free and qemu_malloc usesBlue Swirl2011-08-211-1/+1
| | | | | | | 7267c0947d7e8ae5dff7bafd932c3bc285f43e5c missed a few cases, fix them. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* Update HACKING to refer to g_malloc instead of qemu_mallocAnthony Liguori2011-08-211-3/+3
| | | | Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* HACKING: Update status of format checkingStefan Weil2011-02-251-3/+0
| | | | | | | | | | | | | | | Hopefully all functions with printf like arguments now use format checking. This was tested with default build configuration on linux and windows hosts (including some cross compilations), so chances are good that there remain few (if any) functions without format checking. Therefore the last comment in HACKING is no longer valid but misleading. Cc: Blue Swirl <blauwirbel@gmail.com> Signed-off-by: Stefan Weil <weil@mail.berlios.de> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
* HACKING: add rules for printf-like functionsBlue Swirl2010-09-101-0/+13
| | | | | | Add rules for printf-like functions, based on libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add string management rulesBlue Swirl2010-09-101-0/+24
| | | | | | Add string management rules, somewhat like libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add memory management rulesBlue Swirl2010-09-101-0/+14
| | | | | | Add memory management rules, somewhat like libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add C type rulesBlue Swirl2010-09-101-0/+68
| | | | | | | | | | Add C type rules, adapted from libvirt HACKING. Also include a description of special QEMU scalar types. Move typedef rule from CODING_STYLE rule 3 to HACKING rule 6 where it belongs. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
* HACKING: add preprocessor rulesBlue Swirl2010-09-101-0/+6
Add a new file, HACKING, in order to collect recurring issues with submitted patches. Start with preprocessor rules, adapted from libvirt HACKING. Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
OpenPOWER on IntegriCloud